Committed by
Gerrit Code Review
Bug fix: Graphite reporter
- It never changes report period bacause of Default config - TODO: implement retriever and add some function Change-Id: I578c7fedefbe76f265dd43091588dd19d541ffa1
Showing
2 changed files
with
3 additions
and
3 deletions
... | @@ -127,7 +127,7 @@ public class DefaultGraphiteMetricsReporter implements GraphiteMetricsReporter { | ... | @@ -127,7 +127,7 @@ public class DefaultGraphiteMetricsReporter implements GraphiteMetricsReporter { |
127 | graphiteReporter = buildReporter(graphite); | 127 | graphiteReporter = buildReporter(graphite); |
128 | 128 | ||
129 | try { | 129 | try { |
130 | - graphiteReporter.start(DEFAULT_REPORT_PERIOD, REPORT_TIME_UNIT); | 130 | + graphiteReporter.start(reportPeriod, REPORT_TIME_UNIT); |
131 | } catch (Exception e) { | 131 | } catch (Exception e) { |
132 | log.error("Errors during reporting to graphite, msg: {}" + e.getMessage()); | 132 | log.error("Errors during reporting to graphite, msg: {}" + e.getMessage()); |
133 | } | 133 | } |
... | @@ -154,7 +154,7 @@ public class DefaultGraphiteMetricsReporter implements GraphiteMetricsReporter { | ... | @@ -154,7 +154,7 @@ public class DefaultGraphiteMetricsReporter implements GraphiteMetricsReporter { |
154 | graphiteReporter = buildReporter(graphite); | 154 | graphiteReporter = buildReporter(graphite); |
155 | 155 | ||
156 | try { | 156 | try { |
157 | - graphiteReporter.start(DEFAULT_REPORT_PERIOD, REPORT_TIME_UNIT); | 157 | + graphiteReporter.start(reportPeriod, REPORT_TIME_UNIT); |
158 | } catch (Exception e) { | 158 | } catch (Exception e) { |
159 | log.error("Errors during reporting to graphite, msg: {}" + e.getMessage()); | 159 | log.error("Errors during reporting to graphite, msg: {}" + e.getMessage()); |
160 | } | 160 | } | ... | ... |
... | @@ -18,7 +18,7 @@ package org.onosproject.graphitemetrics; | ... | @@ -18,7 +18,7 @@ package org.onosproject.graphitemetrics; |
18 | import org.onlab.metrics.MetricsReporter; | 18 | import org.onlab.metrics.MetricsReporter; |
19 | 19 | ||
20 | /** | 20 | /** |
21 | - * Interface of graphite reporter that reports all metrics value to graphite server. | 21 | + * A Metric reporter interface for reporting all metrics to graphite server. |
22 | */ | 22 | */ |
23 | public interface GraphiteMetricsReporter extends MetricsReporter { | 23 | public interface GraphiteMetricsReporter extends MetricsReporter { |
24 | } | 24 | } | ... | ... |
-
Please register or login to post a comment