Change in ovirt-engine[master]: core: fix broken functionality for cloning snapshot

shavivi at redhat.com shavivi at redhat.com
Mon Jan 12 11:22:52 UTC 2015


Shahar Havivi has submitted this change and it was merged.

Change subject: core: fix broken functionality for cloning snapshot
......................................................................


core: fix broken functionality for cloning snapshot

Cannot clone to snapshot due to a bug presented in
a8bd1c1e0bcfd1b198ce5b54761ba89bbf384eeb
That commit didn't add the functionality to for adding a new VM id for
snapshot - causing this action to fail with wrong message.

Bug-Url: https://bugzilla.redhat.com/1179145
Change-Id: Ibff3323f31f1458e12d23c9e53a8ff20b97f4621
Signed-off-by: Shahar Havivi <shaharh at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmCommand.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalListModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/CloneVmFromSnapshotModelBehavior.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UserPortalNewVmModelBehavior.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmSnapshotListModel.java
6 files changed, 15 insertions(+), 3 deletions(-)

Approvals:
  Tomas Jelinek: Looks good to me, but someone else must approve
  Shahar Havivi: Verified
  Arik Hadas: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibff3323f31f1458e12d23c9e53a8ff20b97f4621
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list