Change in ovirt-engine[master]: userportal/webadmin: fix missing fields in cloner

ahadas at redhat.com ahadas at redhat.com
Sun Jan 4 15:30:48 UTC 2015


Arik Hadas has submitted this change and it was merged.

Change subject: userportal/webadmin: fix missing fields in cloner
......................................................................


userportal/webadmin: fix missing fields in cloner

Added fields which were missing in the cloneVM part.

The 'edit template-version' feature bypass part of the saving process of
a running edited vm with a different template version(code now ignores
the model data in this scenario because it represents the new
template data which won't be relevant till shutdown). This causes the
saving process to be based solely on VmListModel::preSave() which uses this
cloner as it's only source of vm data.

Change-Id: Id475341c0cb97612927627b0bd5dfca070e19d22
Bug-Url: https://bugzilla.redhat.com/1140569
Signed-off-by: Eldan Shachar <eshachar at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Cloner.java
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Arik Hadas: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id475341c0cb97612927627b0bd5dfca070e19d22
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eldan Shachar <eshachar at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Eldan Shachar <eshachar 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