commit 41c0a31bb771326350c5481256b68f6bccf58a21
Author: Mike Turek <email address hidden>
Date: Thu Jun 15 11:06:46 2017 -0400
Move _abort_attach_volumes functionality to detach_volumes
This is a followup to the Cinder driver patch [0]. There were several
comments during review that asked why we weren't using
detach_volumes instead of _abort_attach_volumes. This patch is a
proposal to remove the method and merge it's functionality with
the detach_volumes method.
This patch also addresses some nits found in the cinder driver.
Reviewed: https:/ /review. openstack. org/467766 /git.openstack. org/cgit/ openstack/ ironic/ commit/ ?id=41c0a31bb77 1326350c5481256 b68f6bccf58a21
Committed: https:/
Submitter: Jenkins
Branch: master
commit 41c0a31bb771326 350c5481256b68f 6bccf58a21
Author: Mike Turek <email address hidden>
Date: Thu Jun 15 11:06:46 2017 -0400
Move _abort_ attach_ volumes functionality to detach_volumes
This is a followup to the Cinder driver patch [0]. There were several attach_ volumes. This patch is a
comments during review that asked why we weren't using
detach_volumes instead of _abort_
proposal to remove the method and merge it's functionality with
the detach_volumes method.
This patch also addresses some nits found in the cinder driver.
[0] b23a176a737f217 e9573d6f10c3059 d22813b159
Change-Id: Ic5584bc1380a65 fdd9a74c174bee6 3bae1c6a4b3
Parital-bug: #1559691