Bug 1562591 - Conf file is missing/incomplete for collectd libvirt plugin
Summary: Conf file is missing/incomplete for collectd libvirt plugin
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: collectd
Version: 13.0 (Queens)
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: beta
: 13.0 (Queens)
Assignee: Matthias Runge
QA Contact: Leonid Natapov
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-04-01 08:29 UTC by Leonid Natapov
Modified: 2018-06-27 13:09 UTC (History)
7 users (show)

Fixed In Version: collectd-5.8.0-8.el7ost
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-06-27 13:08:58 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
RDO 13259 0 None None None 2018-04-04 09:12:43 UTC
Red Hat Product Errata RHEA-2018:2084 0 None None None 2018-06-27 13:09:31 UTC

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


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