From 161adeec849aa15611b15dc2e2afa8ceb6f0073a Mon Sep 17 00:00:00 2001 From: Alexander Chemeris Date: Fri, 8 May 2020 18:19:24 +0300 Subject: stats: Move cfg_stats_interval_cmd() function. cfg_stats_interval_cmd() function was (probably mistakenly) inserted between cfg_stats_reporter_statsd_cmd() and cfg_no_stats_reporter_statsd_cmd() function which makes no sense. Move it below the cfg_no_stats_reporter_log_cmd() to follow the order of the osmo_stats_vty_add_cmds() function calls. Change-Id: I1ecec7025e95cf5ffc21ae3b1c75cf6da8c58de2 --- src/vty/stats_vty.c | 35 +++++++++++++++++------------------ 1 file changed, 17 insertions(+), 18 deletions(-) diff --git a/src/vty/stats_vty.c b/src/vty/stats_vty.c index 296519c3..a512703f 100644 --- a/src/vty/stats_vty.c +++ b/src/vty/stats_vty.c @@ -269,24 +269,6 @@ DEFUN(cfg_stats_reporter_statsd, cfg_stats_reporter_statsd_cmd, return CMD_SUCCESS; } -DEFUN(cfg_stats_interval, cfg_stats_interval_cmd, - "stats interval <1-65535>", - CFG_STATS_STR "Set the reporting interval\n" - "Interval in seconds\n") -{ - int rc; - int interval = atoi(argv[0]); - rc = osmo_stats_set_interval(interval); - if (rc < 0) { - vty_out(vty, "%% Unable to set interval: %s%s", - strerror(-rc), VTY_NEWLINE); - return CMD_WARNING; - } - - return CMD_SUCCESS; -} - - DEFUN(cfg_no_stats_reporter_statsd, cfg_no_stats_reporter_statsd_cmd, "no stats reporter statsd", NO_STR CFG_STATS_STR CFG_REPORTER_STR "Report to a STATSD server\n") @@ -347,6 +329,23 @@ DEFUN(cfg_no_stats_reporter_log, cfg_no_stats_reporter_log_cmd, return CMD_SUCCESS; } +DEFUN(cfg_stats_interval, cfg_stats_interval_cmd, + "stats interval <1-65535>", + CFG_STATS_STR "Set the reporting interval\n" + "Interval in seconds\n") +{ + int rc; + int interval = atoi(argv[0]); + rc = osmo_stats_set_interval(interval); + if (rc < 0) { + vty_out(vty, "%% Unable to set interval: %s%s", + strerror(-rc), VTY_NEWLINE); + return CMD_WARNING; + } + + return CMD_SUCCESS; +} + DEFUN(show_stats, show_stats_cmd, "show stats", -- cgit v1.2.3