Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

CA-86883: Detach SR if SR.destroy fails #1037

Merged
merged 1 commit into from Apr 29, 2013

Conversation

Projects
None yet
3 participants
Contributor

jeromemaloberti commented Feb 21, 2013

CLEARWATER

The storage backend can destroy an SR only if it is attached.
Therefore when xapi wants to destroy an SR, first it attaches it.
When the backend destroys the SR it is no longer attached.

This change adds exception handling so that if the backend fails
to destroy the SR, xapi will detach it.

CA-86883: Detach SR if SR.destroy fails
The storage backend can destroy an SR only if it is attached.
Therefore when xapi wants to destroy an SR, first it attaches it.
When the backend destroys the SR it is no longer attached.

This change adds exception handling so that if the backend fails
to destroy the SR, xapi will detach it.
Member

thomassa commented Feb 21, 2013

This is a mechanically created copy of my pull-request that made the same change in the Master (trunk-ring3) branch: xen-org#1030

malcolmcrossley pushed a commit that referenced this pull request Apr 29, 2013

Merge pull request #1037 from jeromemaloberti/CA-86883-clearwater
CA-86883: Detach SR if SR.destroy fails

@malcolmcrossley malcolmcrossley merged commit aded660 into xapi-project:clearwater Apr 29, 2013

1 check was pending

default Merged build triggered.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment