Change in ovirt-engine[master]: core: Disabled ability to make VM HA on hosted engine

rgolan at redhat.com rgolan at redhat.com
Tue Feb 23 19:09:06 UTC 2016


Roy Golan has submitted this change and it was merged.

Change subject: core: Disabled ability to make VM HA on hosted engine
......................................................................


core: Disabled ability to make VM HA on hosted engine

Disabled checkbox for high availability in the Edit VM
dialog popup and ability to change the setting via the
REST API when the hosted engine option is being used.

These options shouldn't be available on hosted engine
because hosted engine uses a different HA mechanism
than the non-hosted version.

Change-Id: I088cf32361cbd2b59e328b8c31b0e63377865251
Bug-Url: https://bugzilla.redhat.com/1305330
Signed-off-by: Phillip Bailey <phbailey at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/UpdateVmCommandTest.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/EngineMessage.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/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ExistingVmModelBehavior.java
M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.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
9 files changed, 56 insertions(+), 1 deletion(-)

Approvals:
  Jenkins CI: Passed CI tests
  Daniel Erez: Looks good to me, but someone else must approve
  Doron Fediuck: Looks good to me, but someone else must approve
  Roy Golan: Looks good to me, approved
  Greg Sheremeta: Looks good to me, but someone else must approve
  Phillip Bailey: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I088cf32361cbd2b59e328b8c31b0e63377865251
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Phillip Bailey <phbailey at redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Phillip Bailey <phbailey at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list