Change in ovirt-engine[master]: core: don't allow snapshot for suspended vm

ofrenkel at redhat.com ofrenkel at redhat.com
Mon Nov 3 09:27:00 UTC 2014


Omer Frenkel has submitted this change and it was merged.

Change subject: core: don't allow snapshot for suspended vm
......................................................................


core: don't allow snapshot for suspended vm

creating snapshot while vm is suspended is problematic,
since when resuming, vdsm still use the old volumes,
another problem is that the hibernation volumes info is not saved in the
ovf.

Change-Id: Icf2c4c3ec3561df1d35f5e4da964664d63598434
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1120232
Signed-off-by: Omer Frenkel <ofrenkel at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/VmValidator.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/VdcActionUtils.java
3 files changed, 19 insertions(+), 2 deletions(-)

Approvals:
  Omer Frenkel: Verified
  Arik Hadas: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Icf2c4c3ec3561df1d35f5e4da964664d63598434
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list