Ceilometer fails to add proper groups to user

Bug #1603446 reported by MercSniper
22
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Alexey Deryugin
Mitaka
Fix Released
High
Alexey Deryugin

Bug Description

Detailed bug description:
 https://github.com/openstack/fuel-library/blob/stable/mitaka/deployment/puppet/openstack_tasks/manifests/ceilometer/compute.pp

Puppet Manifest creates the ceilometer user on 14.04+UCA compute host but attempts to add user to libvirt group. Per Comment in Github, the libvirt group is supposed to be created during the libvirt-bin group, but either fails to do so or should be changed to libvirtd in manifest.
Steps to reproduce:
 1. Fresh Virtualbox install w/scripts
 2. Create Openstack Cluster using 14.04+UCA
 3. Create a minimum 1 compute+ceph, 1 controller host w/Ceilometer
Expected results:
 No Error
Actual result:
   (/Stage[main]/Ceilometer/User[ceilometer]/groups) change from to libvirt,nova failed: Could not set groups on user[ceilometer]: Execution of '/usr/sbin/usermod -G libvirt,nova ceilometer' returned 6: usermod: group 'libvirt' does not exist
Reproducibility:
 Reset environment, attempt again
Workaround:
 Create a libvirt group on compute node after error
Impact:
 Fail to deploy Cluster and impact new users ability to test Fuel
Description of the environment:
 Operation system: Windows 7 host
 Versions of components: VirtualBox 5.1, Fuel 9.0
 Reference architecture: x64
 Network model: Virtualbox Scripts
 Related projects installed: N/A
Additional Information:
http://pastebin.com/mnKvWqtD

Revision history for this message
MercSniper (mercsniper) wrote :

Another workaround:
/etc/puppet/mitaka-9.0/manifests/ceilometer/compute.pp on fuel-master
-99 groups => ['nova', 'libvirt'],
+99 groups => ['nova', 'libvirtd']

Changed in fuel:
milestone: none → 10.0
importance: Undecided → Medium
status: New → Confirmed
assignee: nobody → Fuel Sustaining (fuel-sustaining-team)
Dmitry Klenov (dklenov)
Changed in fuel:
assignee: Fuel Sustaining (fuel-sustaining-team) → MOS Ceilometer (mos-ceilometer)
importance: Medium → High
tags: added: area-mos
Changed in fuel:
assignee: MOS Ceilometer (mos-ceilometer) → Alexey Deryugin (velovec)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

Fix proposed to branch: master
Review: https://review.openstack.org/343590

Changed in fuel:
status: Confirmed → In Progress
MercSniper (mercsniper)
tags: added: customer-found
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/343590
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=ed074039963e143b14b12faec3b2b170efbbdff8
Submitter: Jenkins
Branch: master

commit ed074039963e143b14b12faec3b2b170efbbdff8
Author: Alexey Deryugin <email address hidden>
Date: Tue Jul 19 15:44:30 2016 +0300

    Fix ceilometer user groups on compute nodes

    Now package [0] handles addition of ceilometer user to libvirt group, so
    shouldn't handle this from Puppet.

    [0] https://review.fuel-infra.org/gitweb?p=openstack-build/ceilometer-build.git;a=blob;f=trusty/debian/ceilometer-common.postinst.in;h=623271b3d33ef4809480c44ac76e3497a34a7e11;hb=refs/heads/master#l7

    Change-Id: I092fbca27db2b0d5324231ded599ddd58b61350a
    Closes-Bug: #1603446

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/356464

Revision history for this message
Alexey Deryugin (velovec) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/mitaka)

Reviewed: https://review.openstack.org/356464
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=8f23ef2d08de2ed1829491e8a6993aee39cb956c
Submitter: Jenkins
Branch: stable/mitaka

commit 8f23ef2d08de2ed1829491e8a6993aee39cb956c
Author: Alexey Deryugin <email address hidden>
Date: Tue Jul 19 15:44:30 2016 +0300

    Fix ceilometer user groups on compute nodes

    Now package [0] handles addition of ceilometer user to libvirt group, so
    shouldn't handle this from Puppet.

    [0] https://review.fuel-infra.org/gitweb?p=openstack-build/ceilometer-build.git;a=blob;f=trusty/debian/ceilometer-common.postinst.in;h=623271b3d33ef4809480c44ac76e3497a34a7e11;hb=refs/heads/master#l7

    Change-Id: I092fbca27db2b0d5324231ded599ddd58b61350a
    Closes-Bug: #1603446
    (cherry picked from commit ed074039963e143b14b12faec3b2b170efbbdff8)

tags: added: area-ceilometer
Revision history for this message
Timur Nurlygayanov (tnurlygayanov) wrote :

The issue is not reproduced om MOS 9.1 environment, marked as Fix Released.

Changed in fuel:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-library 10.0.0rc1

This issue was fixed in the openstack/fuel-library 10.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-library 10.0.0

This issue was fixed in the openstack/fuel-library 10.0.0 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.