Yaniv Bronhaim 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'
> again, metrics is not clear and confusing, we drop oldest samples, each sam
that's where we were wrong. metrics == מדדים. we send the metrics - not samples 
(sample is when you read the data) and not reports (report is when you query 
the data).
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