Bug 1434570

Summary:	Add postgresql collectd plugin on engine machine
Product:	[oVirt] ovirt-engine-metrics	Reporter:	Shirly Radco <sradco>
Component:	RFEs	Assignee:	Shirly Radco <sradco>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Lukas Svaty <lsvaty>
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	1.0.0	CC:	bugs, juwu, lsvaty, ylavi
Target Milestone:	ovirt-4.1.1-1	Keywords:	FutureFeature
Target Release:	1.0.2	Flags:	rule-engine: ovirt-4.1+ rule-engine: exception+ ylavi: planning_ack+ rule-engine: devel_ack+ lsvaty: testing_ack+
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Enhancement
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2017-04-21 09:29:57 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Metrics	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:	1410044
Bug Blocks:	1402901, 1428386

Description Shirly Radco 2017-03-21 19:24:19 UTC

Description of problem:
We would like to be able to monitor the postgresql database performance.
For that we require to add to the collectd configuration file the postgresql plugin.

Comment 1 Red Hat Bugzilla Rules Engine 2017-03-21 19:24:28 UTC

Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

Comment 2 Shirly Radco 2017-03-30 06:35:23 UTC

This patch should be verified by replacing the db credentials manually.

Comment 3 Shirly Radco 2017-03-30 06:38:01 UTC

The db credential should be replaced in the /packaging/setup/ansible/roles/ovirt_collectd/ovirt_engine/templates/builtins-conf-for-ovirt-engine.conf.j2 file before running the ansible script.

Comment 4 Lukas Svaty 2017-03-30 08:47:43 UTC

this needs automatic population of the config, however this is tracked in bug BZ#1434573

this one will be verified with mentioned WA in comment#3

Comment 5 Lukas Svaty 2017-03-30 09:13:46 UTC

meant BZ#1410044 sorry for the mistake

Comment 6 Lukas Svaty 2017-03-30 10:36:50 UTC

due to consistency with bz#1434573 moving to FailedQA after discussion with dev

Comment 7 Red Hat Bugzilla Rules Engine 2017-03-30 10:36:54 UTC

Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

Comment 8 Shirly Radco 2017-04-04 08:35:59 UTC

I added a patch to add the postgresql plugin configuration, but not load it.
So it will not fail the collectd from starting.
Populating and loading the plugin will be don in bug 1429861

Comment 9 Lukas Svaty 2017-04-07 11:31:25 UTC

doing sanity only as this does not add any new functionality

sanity were done that currently playbook runs successfully

verified in ovirt-engine-metrics-1.0.2-1.el7ev.noarch

keep in mind all this does is add configuration for postgresql and does not install any additional rpm or anything else