Bug 1478695 - [RFE] Make collectd interval a parameter
Summary: [RFE] Make collectd interval a parameter
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine-metrics
Classification: oVirt
Component: RFEs
Version: 1.0.5
Hardware: Unspecified
OS: Unspecified
low
medium
Target Milestone: ovirt-4.2.0
: ---
Assignee: Shirly Radco
QA Contact: Lukas Svaty
URL:
Whiteboard:
Depends On: 1495500
Blocks: oVirt-Metrics-and-Logs 1475135 1508921
TreeView+ depends on / blocked
 
Reported: 2017-08-06 08:48 UTC by Shirly Radco
Modified: 2017-12-20 11:11 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2017-12-20 11:11:40 UTC
oVirt Team: Metrics
Embargoed:
rule-engine: ovirt-4.2?
lsvaty: testing_plan_complete-
rule-engine: planning_ack?
rule-engine: devel_ack+
lsvaty: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 82008 0 'None' 'MERGED' 'collectd: add global configurtion role' 2019-11-18 09:20:43 UTC

Description Shirly Radco 2017-08-06 08:48:08 UTC
Description of problem:
Collectd default interval is 10 seconds.
We want to add an collectd_interval parameter that will allow changing the interval as needed when running the metrics ansible.

Comment 1 Yaniv Kaul 2017-08-06 12:00:49 UTC
I think it should be 15 seconds, to match VDSM's interval.

Comment 2 Yaniv Lavi 2017-08-10 11:10:18 UTC
(In reply to Yaniv Kaul from comment #1)
> I think it should be 15 seconds, to match VDSM's interval.

The interval in VDSM is 3 seconds I believe. 15 seconds is for engine pulling.
I think we would be better off with 20 seconds to match the DWH collection rate.

Comment 4 Lukas Svaty 2017-09-25 10:23:22 UTC
(In reply to Yaniv Lavi (Dary) from comment #2)
> (In reply to Yaniv Kaul from comment #1)
> > I think it should be 15 seconds, to match VDSM's interval.
> 
> The interval in VDSM is 3 seconds I believe. 15 seconds is for engine
> pulling.
> I think we would be better off with 20 seconds to match the DWH collection
> rate.

DWH collection sampling was changed to 60 seconds. Want to reconsider?

Comment 5 Shirly Radco 2017-09-28 14:49:03 UTC
DWH is for long term storage and metrics is also for monitoring.
Different needs.

Comment 6 Red Hat Bugzilla Rules Engine 2017-10-10 15:17:27 UTC
Testing has indicated this request is declined. You may appeal this decision by reopening this request.

Comment 7 Lukas Svaty 2017-10-10 15:34:41 UTC
Fixing the mess I made, apologies

Comment 8 Yaniv Kaul 2017-10-15 11:01:18 UTC
Moving back to ASSIGNED as the patch has this comment:
"There is a limitation to setting the interval in /etc/collectd.d/, the plugins that are pre-loaded in /etc/collectd.conf remain with the same interval that is set in that file or the with the default 10 seconds"

Comment 9 Shirly Radco 2017-10-29 12:41:44 UTC
Didi has fixed the issue in centos-opstools.
https://review.rdoproject.org/r/#/c/9802/

Packages needs to be built before this bug can be

Comment 10 Shirly Radco 2017-10-29 12:42:55 UTC
**Packages needs to be built before this bug can be fixed and tested

Comment 11 Shirly Radco 2017-11-02 13:58:30 UTC
This RFE adds a top level configuration role to collectd that allows setting collectd interval with a parameter.

It depends on Didi's patch to collectd packaging.
Until it is available the parameter will be applied to the plugins that are loaded after the file 05-global-configurations.conf.
oVirt plugins are loaded after this file.

Comment 12 Lukas Svaty 2017-12-14 15:32:28 UTC
verified in ovirt-engine-metrics-1.1.1-1.el7ev.noarch

Comment 13 Sandro Bonazzola 2017-12-20 11:11:40 UTC
This bugzilla is included in oVirt 4.2.0 release, published on Dec 20th 2017.

Since the problem described in this bug report should be
resolved in oVirt 4.2.0 release, published on Dec 20th 2017, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.


Note You need to log in before you can comment on or make changes to this bug.