recent change to qemu.conf breaks some deployments

Bug #1572819 reported by Mark Casey
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
kolla
Fix Released
Critical
Steven Dake
Liberty
Fix Released
Critical
Steven Dake
Mitaka
Fix Released
Critical
Steven Dake

Bug Description

The Dockerfile template for the nova_libvirt container has a sed one-liner intended to set the parameter "stdio_handler" to equal "file" (https://github.com/openstack/kolla/blob/stable/mitaka/docker/nova/nova-libvirt/Dockerfile.j2#L32).

This is done so that libvirt operates like it is less than version 1.3.0 and does not look for a new daemon that was introduced in 1.3.0. This is what happens when 1.3.0 is installed without this option: https://bugzilla.redhat.com/show_bug.cgi?id=1290357

The option was added to Kolla here: https://review.openstack.org/#/c/279910/

However from what I'm seeing this is not being set because the template for /etc/libvirt/qemu.conf is currently empty, which for me at least is causing the "libvirtError: Failed to connect socket to '/var/run/libvirt/virtlogd-sock': No such file or directory
" error mentioned in the redhat bug linked above.

Assuming this assessment is correct the nova_libvirt Dockerfile template should be changed to set "stdio_handler" as desired if the option is present BUT also add the desired line if there is not existing value.

some info on the environment in which I'm seeing this:
branch in use: stable/mitaka
distro: Ubuntu 15.10
install type: source
multinode
KVM enabled / non qemu

Revision history for this message
Mark Casey (mark-casey) wrote :

I forgot to mention that if I use ansible to add this option on my target nodes and restart nova_libvirt container I am then able to start VMs without error.

ansible -i /usr/local/share/kolla/ansible/inventory/multinode -m shell -a 'echo "stdio_handler = \"file\"" >> /etc/kolla/nova-libvirt/qemu.conf' all

Steven Dake (sdake)
Changed in kolla:
importance: Undecided → Critical
summary: - nova_libvirt's Dockerfile.j2 has a sed one-liner which does not work
- because the target file is empty
+ recent change to qemu.conf breaks some deployments
Changed in kolla:
status: New → Confirmed
assignee: nobody → Steven Dake (sdake)
milestone: none → newton-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to kolla (stable/mitaka)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on kolla (stable/mitaka)

Change abandoned by Steven Dake (<email address hidden>) on branch: stable/mitaka
Review: https://review.openstack.org/308780
Reason: error in workflow

Changed in kolla:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kolla (master)

Reviewed: https://review.openstack.org/308749
Committed: https://git.openstack.org/cgit/openstack/kolla/commit/?id=a861450fd29d7a2ea56eb755473da171443b38f6
Submitter: Jenkins
Branch: master

commit a861450fd29d7a2ea56eb755473da171443b38f6
Author: Steven Dake <email address hidden>
Date: Thu Apr 21 01:08:04 2016 -0400

    Fix failure to deploy with qemu.conf

    Closes-Bug: #1572819

    Change-Id: I1f2edb482af1e85708159a0dba44a79427dc5af3

Changed in kolla:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on kolla (stable/mitaka)

Change abandoned by Michal Rostecki (<email address hidden>) on branch: stable/mitaka
Review: https://review.openstack.org/308780

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kolla (stable/mitaka)

Reviewed: https://review.openstack.org/308780
Committed: https://git.openstack.org/cgit/openstack/kolla/commit/?id=81f016bd3b9023faeadd25409a5a5005dba80f01
Submitter: Jenkins
Branch: stable/mitaka

commit 81f016bd3b9023faeadd25409a5a5005dba80f01
Author: Steven Dake <email address hidden>
Date: Thu Apr 21 01:08:04 2016 -0400

    Fix failure to deploy with qemu.conf

    Closes-Bug: #1572819

    Change-Id: I1f2edb482af1e85708159a0dba44a79427dc5af3
    (cherry picked from commit a861450fd29d7a2ea56eb755473da171443b38f6)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kolla (stable/liberty)

Reviewed: https://review.openstack.org/308820
Committed: https://git.openstack.org/cgit/openstack/kolla/commit/?id=abf22e991c6d9ac14bbc4714b3bacd9238213761
Submitter: Jenkins
Branch: stable/liberty

commit abf22e991c6d9ac14bbc4714b3bacd9238213761
Author: Steven Dake <email address hidden>
Date: Thu Apr 21 01:08:04 2016 -0400

    Fix failure to deploy with qemu.conf

    Closes-Bug: #1572819

    Change-Id: I1f2edb482af1e85708159a0dba44a79427dc5af3
    (cherry picked from commit a861450fd29d7a2ea56eb755473da171443b38f6)

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/kolla 2.0.0

This issue was fixed in the openstack/kolla 2.0.0 release.

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/kolla 1.1.0

This issue was fixed in the openstack/kolla 1.1.0 release.

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/kolla 3.0.0.0b1

This issue was fixed in the openstack/kolla 3.0.0.0b1 development milestone.

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.