Change in ovirt-engine[master]: core: Make disk list of VM editable on HE update

rgolan at redhat.com rgolan at redhat.com
Thu Apr 21 10:09:54 UTC 2016


Roy Golan has submitted this change and it was merged.

Change subject: core: Make disk list of VM editable on HE update
......................................................................


core: Make disk list of VM editable on HE update

Disks are never updated by the UpdateVmCommand but the editable field
checks are checking the value if it changed. The UI will not send the
disk list with the VM when it updates a VM but the REST API does.

Mark the diskList field as editable to make updating the Hosted Engine
always work.

Change-Id: I1ed00104d929b71de0ceb6d6cce9482946a8f5ab
Bug-Url: https://bugzilla.redhat.com/1326810
Signed-off-by: Roman Mohr <rmohr at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmBase.java
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Shahar Havivi: Looks good to me, but someone else must approve
  Roman Mohr: Verified
  Jenkins CI: Passed CI tests
  Roy Golan: Looks good to me, approved
  Arik Hadas: Looks good to me, but someone else must approve
  Jenny Tokar: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1ed00104d929b71de0ceb6d6cce9482946a8f5ab
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list