OpenStack Compute (nova)

libvirt does not store connection_info after BFV setup

Bug #1328245 reported by Dan Smith
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
Dan Smith
OpenStack Compute (nova) 2014.2 "juno"
Havana
Fix Released
Medium
Dan Smith
OpenStack Compute (nova) 2013.2.4
Icehouse
Fix Released
Medium
Dan Smith
OpenStack Compute (nova) 2014.1.2

Bug Description

If booting from a volume, the virt driver does the setup of the volume with cinder before starting the instance. This differs from the attach volume case, which is managed by nova itself. Since the connect operation could yield new details in the connection_info structure that need to be persisted until teardown time, it is important that the connection_info be written back after connect completes. Nova's attach_volume() does this, but libvirt does not. Specifically in the case of the fibre channel code, this means we don't persist information about multipath devices which means we don't fully tear down everything at disconnect time.

This is present in at least Havana, and I expect it is present in Icehosue and master as well.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to nova (stable/havana)

Related fix proposed to branch: stable/havana
Review: https://review.openstack.org/98874

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to nova (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/99699

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to nova (master)

Reviewed: https://review.openstack.org/99699
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=a1a2d0605b555f149846eb7e74f7d1760e55ed75
Submitter: Jenkins
Branch: master

commit a1a2d0605b555f149846eb7e74f7d1760e55ed75
Author: Dan Smith <email address hidden>
Date: Thu Jun 12 08:56:21 2014 -0700

    Save connection info in libvirt after volume connect

    The libvirt driver will connect volumes for BFV while processing
    the guest configuration, but not save the connection_info into
    the database. This adds a save there to make sure that any
    storage driver details get persisted.

    Change-Id: Ia99e04ad80a2823a6852debf0919e72b2f93bd26
    Related-bug: #1328245

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to nova (stable/icehouse)

Related fix proposed to branch: stable/icehouse
Review: https://review.openstack.org/100556

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to nova (stable/havana)

Reviewed: https://review.openstack.org/98874
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=a2aca5c9fb2afaa2f66cefc6cf19c658e6f44430
Submitter: Jenkins
Branch: stable/havana

commit a2aca5c9fb2afaa2f66cefc6cf19c658e6f44430
Author: Dan Smith <email address hidden>
Date: Fri May 23 07:05:52 2014 -0700

    Save connection info in libvirt after volume connect

    The libvirt driver will connect volumes for BFV while processing
    the guest configuration, but not save the connection_info into
    the database. This adds a save there to make sure that any
    storage driver details get persisted.

    Related-bug: #1328245

    Change-Id: Idd6ef94ba2129b65e685a0435bc79f213d6f3f04

tags: added: in-stable-havana
Alan Pevec (apevec)
tags: removed: in-stable-havana
Changed in nova:
status: Confirmed → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to nova (stable/icehouse)

Reviewed: https://review.openstack.org/100556
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=a365897a32714e37c0b190594c0bdc827adb36f9
Submitter: Jenkins
Branch: stable/icehouse

commit a365897a32714e37c0b190594c0bdc827adb36f9
Author: Dan Smith <email address hidden>
Date: Thu Jun 12 08:56:21 2014 -0700

    Save connection info in libvirt after volume connect

    The libvirt driver will connect volumes for BFV while processing
    the guest configuration, but not save the connection_info into
    the database. This adds a save there to make sure that any
    storage driver details get persisted.

    Change-Id: Ia99e04ad80a2823a6852debf0919e72b2f93bd26
    Related-bug: #1328245
    (cherry picked from commit a1a2d0605b555f149846eb7e74f7d1760e55ed75)

tags: added: in-stable-icehouse
Russell Bryant (russellb)
Changed in nova:
milestone: none → juno-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: juno-2 → 2014.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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