Harald Welte has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/11503 )

Change subject: make 'show rate-counters' show group index
......................................................................

make 'show rate-counters' show group index

Show each rate counter group's index in the output of the
'show rate-counters' command, to provide some way of telling
apart distinct instances of the same rate counter group.

This is not a very user-friendly UI because these indices are
generated internally by libosmocore and/or applications, so
users cannot easily assign meaning to these indices. However,
the current rate counter implementation doesn't allow for more.

Change-Id: Ieb151239407e4b2f8859fefec8d0670f5ddf908a
Related: OS#3674
---
M src/vty/stats_vty.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Jenkins Builder: Verified
  Harald Welte: Looks good to me, approved



diff --git a/src/vty/stats_vty.c b/src/vty/stats_vty.c
index 62153f3..c911087 100644
--- a/src/vty/stats_vty.c
+++ b/src/vty/stats_vty.c
@@ -533,7 +533,7 @@
 static int rate_ctr_group_handler(struct rate_ctr_group *ctrg, void *sctx_)
 {
        struct vty *vty = sctx_;
-       vty_out(vty, "%s:%s", ctrg->desc->group_description, VTY_NEWLINE);
+       vty_out(vty, "%s %u:%s", ctrg->desc->group_description, ctrg->idx, 
VTY_NEWLINE);
        vty_out_rate_ctr_group_fmt(vty, "%25n: %10c (%S/s %M/m %H/h %D/d) %d", 
ctrg);
        return 0;
 }

--
To view, visit https://gerrit.osmocom.org/11503
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ieb151239407e4b2f8859fefec8d0670f5ddf908a
Gerrit-Change-Number: 11503
Gerrit-PatchSet: 1
Gerrit-Owner: Stefan Sperling <ssperl...@sysmocom.de>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Jenkins Builder (1000002)

Reply via email to