Change in ovirt-engine[master]: core: migrating-to is a running state as well
ofrenkel at redhat.com
ofrenkel at redhat.com
Tue Sep 22 10:16:05 UTC 2015
Omer Frenkel has submitted this change and it was merged.
Change subject: core: migrating-to is a running state as well
......................................................................
core: migrating-to is a running state as well
mirgatingTo was a status that wasn't in use.
we started to use the migratingTo status during the migration process.
and many places checking for running() not taking into consideration the
migratingTo, although it should, just as migratingFrom.
in a specific scenario, user was able to remove the vm while it was
migrating to, and caused the vm to become external.
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1121089
Change-Id: I39403915fa17439224176fc4e5c065311142485a
Signed-off-by: Omer Frenkel <ofrenkel at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VMStatus.java
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Jenkins CI: Passed CI tests
Omer Frenkel: Verified
Arik Hadas: Looks good to me, approved
Michal Skrivanek: Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/31204
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I39403915fa17439224176fc4e5c065311142485a
Gerrit-PatchSet: 4
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: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
More information about the Engine-commits
mailing list