Change in ovirt-engine[master]: core: infrastructure for VmDevices in next run VM snapshot

derez at redhat.com derez at redhat.com
Mon Jul 14 17:45:51 UTC 2014


Daniel Erez has submitted this change and it was merged.

Change subject: core: infrastructure for VmDevices in next run VM snapshot
......................................................................


core: infrastructure for VmDevices in next run VM snapshot

Added an infrastructure for including VmDevices in a snapshot
of type 'Next Run'. This should allow enabling/disabling of
devices while VM is running (i.e. similarly to VM properties that
are editable on a non-running VM, a warning message [1] is being
shown upon editing a VM device status).

* Created a new annotation interface for devices:
  'EditableDeviceOnVmStatusField'.
* VmHandler - initialized mUpdateVmsStatic with those fields.
* UpdateVmCommand - passed a list of devices for next run
  when creating the snapshot with new configuration.
* GetVmUpdatesOnNextRunExistsParameters now gets an instance of
  VmManagementParametersBase in ctr [since the new annotation
  (EditableDeviceOnVmStatusField) will be used in this
  parameters file].
* GetVmUpdatesOnNextRunExistsQuery - added a validation for
  updating VM devices.
* Modified client invocations accordingly.

[1] "Some of the changes will be applied after the NEXT RESTART"

Change-Id: I4e7add4e7e09fe8d6d1f93f0d2b96795d838efa2
Bug-Url: https://bugzilla.redhat.com/1035289
Signed-off-by: Daniel Erez <derez at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetVmUpdatesOnNextRunExistsQuery.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsHandler.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmTemplateHandler.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/SnapshotsManager.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/VmDeviceUtils.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/backendinterfaces/BaseHandler.java
A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/EditableDeviceOnVmStatusField.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetVmUpdatesOnNextRunExistsParameters.java
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ObjectIdentityChecker.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.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/VmListModel.java
15 files changed, 289 insertions(+), 104 deletions(-)

Approvals:
  Daniel Erez: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4e7add4e7e09fe8d6d1f93f0d2b96795d838efa2
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel 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