Omer Frenkel has uploaded a new change for review. 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 <ofren...@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(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/04/31204/1 diff --git a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VMStatus.java b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VMStatus.java index dd19052..3d9a2e0 100644 --- a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VMStatus.java +++ b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VMStatus.java @@ -96,7 +96,7 @@ */ public boolean isRunning() { return this == Up || this == PoweringDown - || this == PoweringUp || this == MigratingFrom || this == WaitForLaunch || this == RebootInProgress; + || this == PoweringUp || this == MigratingFrom || this == MigratingTo || this == WaitForLaunch || this == RebootInProgress; } /** -- To view, visit http://gerrit.ovirt.org/31204 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I39403915fa17439224176fc4e5c065311142485a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Omer Frenkel <ofren...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
