Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1562591

Summary:	Conf file is missing/incomplete for collectd libvirt plugin
Product:	Red Hat OpenStack	Reporter:	Leonid Natapov <lnatapov>
Component:	collectd	Assignee:	Matthias Runge <mrunge>
Status:	CLOSED ERRATA	QA Contact:	Leonid Natapov <lnatapov>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	13.0 (Queens)	CC:	jbadiapa, jschluet, lars, mmagr, mrunge, rmccabe, sradco
Target Milestone:	beta	Keywords:	Reopened, Triaged
Target Release:	13.0 (Queens)
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	collectd-5.8.0-8.el7ost	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2018-06-27 13:08:58 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Leonid Natapov 2018-04-01 08:29:51 UTC

Description of problem:

libvirt plugin fails to load because data is missing in the plugin conf file

[2018-04-01 08:02:52] virt plugin: Unable to connect: virConnectOpen failed.
[2018-04-01 08:02:52] Initialization of plugin `virt' failed with status -1. Plugin will be unloaded.
[2018-04-01 08:02:52] plugin_unregister_read: No such read function: virt





libvirt.conf plugin looks like this:

--------
()[root@controller-0 collectd.d]# cat libvirt.conf
LoadPlugin libvirt
---------

We should either to provide complete libvirt.conf file or to remove it at tall and let user to create it. Now,we do provide conf file but it misses required data which is wrong.

Comment 1 RHEL Program Management 2018-04-01 08:33:00 UTC

Quality Engineering Management has reviewed and declined this request.
You may appeal this decision by reopening this request.

Comment 2 Leonid Natapov 2018-04-04 08:28:15 UTC

Re opening this bug. qa_ack was set to minus by mistake.

Comment 3 Matthias Runge 2018-04-04 08:56:15 UTC

In general, the config would be overwritten by puppet-collectd, thus one can provide the connection string via config.yaml.

I'll propose a patch upstream in a few mins.

Comment 6 Matthias Runge 2018-04-04 12:45:16 UTC

merged upstream

Comment 7 Shirly Radco 2018-04-16 08:59:32 UTC

Our oVirt metrics conf file is like this:

LoadPlugin virt
<Plugin virt>
    ExtraStats "disk disk_err pcpu vcpupin cpu_util domain_state"
    HostnameFormat hostname
    PluginInstanceFormat name
</Plugin>

We get the same error...
Is there something missing?

Comment 8 Matthias Runge 2018-04-16 09:12:46 UTC

This bug is about a missing config file for the virt plugin. Ok, the config only contains LoadPlugin virt, and nothing more. This causes the plugin to be unloaded.

Shirly, I believe you need to specify the connection URL, which seems to be missing in your snippet. If you receive another error message, please open a new bug rather than posting to another one. Opening a new bug will help us to guarantee that both issues are fixed. Thank you.

Comment 12 Leonid Natapov 2018-05-06 09:40:01 UTC

conf file updated.

LoadPlugin libvirt

<Plugin "virt">
  Connection "qemu:///system"
  HostnameFormat "hostname uuid"
  ExtraStats "disk disk_err domain_state pcpu perf vcpupin cpu_util"
</Plugin>

Comment 14 errata-xmlrpc 2018-06-27 13:08:58 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/RHEA-2018:2084