Change in ovirt-engine[master]: engine: Disallow cross-DC template creation

ofrenkel at redhat.com ofrenkel at redhat.com
Thu Jun 19 14:52:29 UTC 2014


Omer Frenkel has submitted this change and it was merged.

Change subject: engine: Disallow cross-DC template creation
......................................................................


engine: Disallow cross-DC template creation

Now we check during creation of template whether the target cluster
lies on the same storage domain as the source VM even if the template
contains no disks.

Change-Id: I6d9943ce1fb3736f280aaecc93896fc174c6b7dd
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1098951
Signed-off-by: Martin Betak <mbetak at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateCommand.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java
M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java
M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
6 files changed, 13 insertions(+), 4 deletions(-)

Approvals:
  Martin Betak: Verified
  Omer Frenkel: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6d9943ce1fb3736f280aaecc93896fc174c6b7dd
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list