Bug 1745510

Summary: [RFE] Implement configuration logic for collectd-sensubility
Product: Red Hat OpenStack Reporter: Martin Magr <mmagr>
Component: puppet-tripleoAssignee: Martin Magr <mmagr>
Status: CLOSED ERRATA QA Contact: Leonid Natapov <lnatapov>
Severity: high Docs Contact:
Priority: high    
Version: 16.0 (Train)CC: amoralej, csibbitt, jjoyce, jschluet, lhh, lnatapov, mburns, mgarciac, mrunge, nwolf, oblaut, shrjoshi, slinaber, tvignaud
Target Milestone: z2Keywords: FutureFeature, Triaged
Target Release: 16.0 (Train on RHEL 8.1)   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: 1733257
: 1803778 (view as bug list) Environment:
Last Closed: 2020-05-14 12:15:28 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1733257    
Bug Blocks: 1744122, 1803778    

Comment 7 Martin Magr 2019-11-12 11:17:35 UTC
Test instructions:

Deploy collectd as you are used to with additional configuration:
1. add "CollectdEnableSensubility: true" to your environment file to enable sensubility
2. point sensubility to Sensu server side via "CollectdSensubilityConnection" in environment file
3. modify any default configuration if required (for documentation see [1])

[1] https://github.com/openstack/tripleo-heat-templates/blob/master/deployment/metrics/collectd-container-puppet.yaml#L304,L378

Comment 8 Leonid Natapov 2019-11-18 14:31:58 UTC
Failed QA. collectd sensubility container is missing.

Comment 9 Leonid Natapov 2019-11-18 14:51:52 UTC
Please,ignore comment #8. 

What we need is collectd sensubility to be part of the existing collectd image.

Comment 10 Martin Magr 2019-12-03 14:00:23 UTC
We will need to enable collectd-sensubility in collectd container downstream. Everything is ready upstream in kolla. I believe there was collectd-sensubility downstream pull BZ. Do you guys need a separate BZ for the inclusion in collectd container?

Comment 11 Martin Magr 2019-12-03 15:48:57 UTC
Added additional patches to enable collectd-sensubility access to all commands to enable implementation various functionality in checks which sensu-client has.

Comment 12 Matthias Runge 2019-12-04 09:26:19 UTC
IMU, kolla switched away from centos opstools until Centos 8 is available.

Comment 13 Martin Magr 2019-12-09 17:01:49 UTC
I still see collectd-sensubility on the plugin list [1] and centos-opstools is the only place from where they can get the RPM. The important info from comment #10 is that RelDel have a place from where they can pull the package downstream.


[1] https://github.com/openstack/kolla/blob/master/docker/collectd/Dockerfile.j2#L37

Comment 14 Martin Magr 2020-02-04 13:02:50 UTC
Guys, any chance we can get collectd-sensubility inside the collectd container?

Comment 15 Matthias Runge 2020-02-21 11:11:08 UTC
moving back to ON_DEV, since there are more patches to be backported

Comment 16 Chris Sibbitt 2020-03-04 15:22:02 UTC
Adjusted target milestone to match https://bugzilla.redhat.com/show_bug.cgi?id=1744122 after talking to Martin

Comment 25 Leonid Natapov 2020-05-12 16:10:28 UTC
Verified. collectd-sensubility implemented.

Comment 27 errata-xmlrpc 2020-05-14 12:15:28 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

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

https://access.redhat.com/errata/RHBA-2020:2114