Tal Nisan has submitted this change and it was merged.
Change subject: core: Avoid errors when applying NEXT_RUN snapshot
......................................................................
core: Avoid errors when applying NEXT_RUN snapshot
* Ignore changes in clusterCompatibilityVersionOrigin. It is not stored
in OVF, so will never be changed when a snapshot is applied.
* Do not check CPU Profile changing permissons, if cpuProfileId is not
changed. Current user is null in ProcessDownVmCommand, so this
permission check will always fail. But cpuProfileId is not stored in
OVF, so will never be changed when a snapshot is applied and there is
no need to check permissions.
Change-Id: Ia2fd9ca872ab1c438d5f02a98c55ab91991cb11a
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1313369
Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com>
---
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/snapshots/SnapshotsManager.java
2 files changed, 2 insertions(+), 1 deletion(-)
Approvals:
Tomas Jelinek: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit
https://gerrit.ovirt.org/54965
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia2fd9ca872ab1c438d5f02a98c55ab91991cb11a
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6.4
Gerrit-Owner: Shmuel Leib Melamud <smelamud(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>