Cinder leaves volume mapped when copying fails

Bug #1880971 reported by Gorka Eguileor
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Gorka Eguileor

Bug Description

If we fail to do the local attach after we have mapped a volume during generic cinder offline migration and generic revert to snapshot we end up leaving the volume mapped to the host.

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

Fix proposed to branch: master
Review: https://review.opendev.org/731260

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

Reviewed: https://review.opendev.org/731260
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=669aee29450be06632e31b5f1f9ebba90e522a4c
Submitter: Zuul
Branch: master

commit 669aee29450be06632e31b5f1f9ebba90e522a4c
Author: Gorka Eguileor <email address hidden>
Date: Wed May 27 17:09:12 2020 +0200

    Fix leave volume mapped on attach failure

    If we fail to do the local attach after we have mapped a volume during
    generic cinder offline migration and generic revert to snapshot we end
    up leaving the volume mapped to the host.

    This is because the call to _connect_device is not included in the
    try...except clause we have to remove the mapping as well as the fact
    that we are always trying to do the local detach when we clean up.

    Closes-Bug: #1880971
    Change-Id: I5319bdaa8f7e468300729dbda358ed0329dc19ad

Changed in cinder:
status: In Progress → Fix Released
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.