Change in ovirt-engine[ovirt-engine-4.0]: core: Validate images existance for unregistered Template.

tnisan at redhat.com tnisan at redhat.com
Tue Sep 13 14:03:09 UTC 2016


Tal Nisan has submitted this change and it was merged.

Change subject: core: Validate images existance for unregistered Template.
......................................................................


core: Validate images existance for unregistered Template.

Once an unregister Template entity is being registered to the engine,
the engine should verify all its disks existance in the Storage.
The reason the engine should perform this validation is since the
Template's OVF might not be synced with the disks in the Storage.

If one of the disks doesn't exists, the engine should block
the operation.
This behavior should be compatible with the behavior being performed
today once a Template is being imported from an export domain.

Change-Id: I3b890bd831c34770d6fd43416d5b15888b45f85b
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
Bug-Url: https://bugzilla.redhat.com/1367488
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ImportVmTemplateFromConfigurationCommand.java
1 file changed, 5 insertions(+), 0 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3b890bd831c34770d6fd43416d5b15888b45f85b
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini 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: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list