SMBFS volume attach race condition

Bug #1461081 reported by Lucian Petrut
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Undecided
Lucian Petrut
Kilo
Fix Released
Undecided
Unassigned

Bug Description

When the SMBFS volume backend is used and a volume is detached, the according SMB share is detached if no longer used.

This can cause issues if at the same time, a different volume stored on the same share is being attached as the according disk image will not be available.

This affects the Libvirt driver as well as the Hyper-V one. In case of Hyper-V, the issue can easily be fixed by using the share path as a lock when performing attach/detach volume operations.

Trace: http://paste.openstack.org/show/256096/

description: updated
Changed in nova:
assignee: nobody → Lucian Petrut (petrutlucian94)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

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

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

commit 5aac3cd675af87b053e1a69d822b5f3bfdab5257
Author: Lucian Petrut <email address hidden>
Date: Tue Jun 2 18:52:23 2015 +0300

    Hyper-V: Fix SMBFS volume attach race condition

    When the SMBFS volume backend is used and a volume is detached,
    the according SMB share is unmounted if no longer used.

    This can cause issues if at the same time, a different volume
    stored on the same share is being attached as the according
    disk image will not be available.

    This patch fixes the issue by synchronizing volume attach and share
    unmount operations based on the share path.

    Closes-Bug: #1461081

    Change-Id: I45f202ad6ada0c472759e8ba3c122f3cfbeee7cc

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → liberty-2
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/kilo)

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/212483

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

Reviewed: https://review.openstack.org/212483
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=07506f55c383e01b887f6f08b265fd519e543510
Submitter: Jenkins
Branch: stable/kilo

commit 07506f55c383e01b887f6f08b265fd519e543510
Author: Lucian Petrut <email address hidden>
Date: Tue Jun 2 18:52:23 2015 +0300

    Hyper-V: Fix SMBFS volume attach race condition

    When the SMBFS volume backend is used and a volume is detached,
    the according SMB share is unmounted if no longer used.

    This can cause issues if at the same time, a different volume
    stored on the same share is being attached as the according
    disk image will not be available.

    This patch fixes the issue by synchronizing volume attach and share
    unmount operations based on the share path.

    Closes-Bug: #1461081

    Change-Id: I45f202ad6ada0c472759e8ba3c122f3cfbeee7cc
    (cherry picked from commit 5aac3cd675af87b053e1a69d822b5f3bfdab5257)

tags: added: in-stable-kilo
Thierry Carrez (ttx)
Changed in nova:
milestone: liberty-2 → 12.0.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (master)

Change abandoned by Lucian Petrut (<email address hidden>) on branch: master
Review: https://review.openstack.org/187619
Reason: This change I7b7ad794b1e15dcd63344e01a520d6c595a73d70 will fix the issue in a more generic way, helping other remote fs based volume drivers as well.

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (master)

Change abandoned by Sean Dague (<email address hidden>) on branch: master
Review: https://review.openstack.org/383860
Reason: This review is > 6 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Sean Dague (<email address hidden>) on branch: master
Review: https://review.openstack.org/290510
Reason: This review is > 6 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

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.