Change in ovirt-engine[master]: core: Set image to illegal in storage while uploading.

mlipchuk at redhat.com mlipchuk at redhat.com
Sun Apr 24 08:21:28 UTC 2016


Maor Lipchuk has submitted this change and it was merged.

Change subject: core: Set image to illegal in storage while uploading.
......................................................................


core: Set image to illegal in storage while uploading.

Currently, when uploading an image- only the engine side sets this image
to locked/illegal- preventing any usage of that image.

However, the image is marked as legal in the storage. regardless to
having the actual image not synced with its engine's representation-
this could break the image's state in the engine as well when detaching
the image's containing domain, and importing it back.

When setting the image to illegal in the storage- importing it will make
the image illegal in the engine when importing again.

This patch makes the image marked as illegal in the storage while the
image is being uploaded.

Change-Id: Id3551b32d4d5f7c5a23de58687612890fcf8cd29
Signed-off-by: Amit Aviram <aaviram at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/UploadImageCommand.java
1 file changed, 26 insertions(+), 0 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Amit Aviram: Verified
  Maor Lipchuk: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id3551b32d4d5f7c5a23de58687612890fcf8cd29
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Amit Aviram <aaviram at redhat.com>
Gerrit-Reviewer: Amit Aviram <aaviram at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list