Change in ovirt-engine[ovirt-engine-3.6]: core: Catch SC_NOT_FOUND status on cinder disk remove

tnisan at redhat.com tnisan at redhat.com
Thu Oct 8 13:10:17 UTC 2015


Tal Nisan has submitted this change and it was merged.

Change subject: core: Catch SC_NOT_FOUND status on cinder disk remove
......................................................................


core: Catch SC_NOT_FOUND status on cinder disk remove

Add catch for OpenStackResponseException on getDiskStatus once we got SC_NOT_FOUND
status.
The disk, which was being removed, might already get removed when we
check the status so there is no reason to mark it as illegal.
also, fix the error log by replacing brackets to valid ones.

Change-Id: Id576359151e7e617d926e4d2dd83f32baa91625a
Bug-Url: https://bugzilla.redhat.com/1269439
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RemoveCinderDiskCommandCallback.java
1 file changed, 18 insertions(+), 4 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Allon Mureinik: Looks good to me, approved
  Maor Lipchuk: Verified



-- 
To view, visit https://gerrit.ovirt.org/47125
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id576359151e7e617d926e4d2dd83f32baa91625a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list