Nir Soffer has posted comments on this change. Change subject: Modify reports to metrics ......................................................................
Patch Set 2: (3 comments) https://gerrit.ovirt.org/#/c/60450/2/lib/vdsm/config.py.in File lib/vdsm/config.py.in: Line 446: ('collector_address', 'localhost', Line 447: 'Metric collector address (default localhost)'), Line 448: Line 449: ('collector_type', 'statsd', Line 450: 'Metric collector type (supporting statsd or hawkular)'), > I should use always metrics Automatic replacement of reports with metrics is wrong, do this. Line 451: Line 452: ('queue_size', '100', Line 453: 'Number of metrics to queue if collector is not responsive.' Line 454: ' When the queue is full, oldest metrics are dropped. Used only' Line 449: ('collector_type', 'statsd', Line 450: 'Metric collector type (supporting statsd or hawkular)'), Line 451: Line 452: ('queue_size', '100', Line 453: 'Number of metrics to queue if collector is not responsive.' > nope.. that's exactly the point. metrics is not clear - we queue metric reports. If you think report may be confusing, use metric samples. Line 454: ' When the queue is full, oldest metrics are dropped. Used only' Line 455: ' by hawkular-client collector (default 100).'), Line 456: ]), Line 457: Line 450: 'Metric collector type (supporting statsd or hawkular)'), Line 451: Line 452: ('queue_size', '100', Line 453: 'Number of metrics to queue if collector is not responsive.' Line 454: ' When the queue is full, oldest metrics are dropped. Used only' > why? "oldest metrics are dropped" is the right term again, metrics is not clear and confusing, we drop oldest samples, each sample contains many metrics. Line 455: ' by hawkular-client collector (default 100).'), Line 456: ]), Line 457: Line 458: # Section: [devel] -- To view, visit https://gerrit.ovirt.org/60450 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib0aa10042b1c50fbdd94ae69053c74b57c3e6f40 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Irit Goihman <igoih...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Yaniv Kaul <yk...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org