Change in ovirt-engine[master]: core: there is no validation on domain status when deleting ...

mkublin at redhat.com mkublin at redhat.com
Thu Jun 21 08:47:17 UTC 2012


Michael Kublin has submitted this change and it was merged.

Change subject: core: there is no validation on domain status when deleting snapshot (#834021)
......................................................................


core: there is no validation on domain status when deleting snapshot (#834021)

https://bugzilla.redhat.com/834021

The bug is not correct a validation does exist, but in order to validate a call to vdsm is send,
it is not needed for that case, because the engines knows that status of domain at DB is maintaince.
So, in order to solve a problem I change order of checks

Change-Id: I5fc120f5fffbad125b71b0d8339484f37bd016b1
Signed-off-by: Michael Kublin <mkublin at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImagesHandler.java
1 file changed, 12 insertions(+), 11 deletions(-)

Approvals:
  Michael Kublin: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/5571
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5fc120f5fffbad125b71b0d8339484f37bd016b1
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list