Change in ovirt-engine[ovirt-engine-3.6]: core: migrating-to is a running state as well

piotr.kliczewski at gmail.com piotr.kliczewski at gmail.com
Tue Sep 22 13:00:38 UTC 2015


Piotr Kliczewski 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>
(cherry picked from commit 9b7f8309f7d4446fcf2f83fb87bd158244bf2a5f)
---
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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I39403915fa17439224176fc4e5c065311142485a
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list