Nir Soffer has posted comments on this change.

Change subject: Modify reports to metrics
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.ovirt.org/#/c/60450/2/lib/vdsm/config.py.in
File lib/vdsm/config.py.in:

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'
> that's where we were wrong. metrics == מדדים. we send the metrics - not sam
We sample the system and report bunch of metrics on each sampling cycle. We 
queue groups of metrics, not single metrics. Make the comment clear about that.
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

Reply via email to