Change in ovirt-engine[master]: core: Make VmAnalyzer to treat a migrated Paused VM as success

Code Review gerrit at ovirt.org
Sun Jul 2 09:03:24 UTC 2017


>From Arik Hadas <ahadas at redhat.com>:

Arik Hadas has submitted this change and it was merged. ( https://gerrit.ovirt.org/78305 )

Change subject: core: Make VmAnalyzer to treat a migrated Paused VM as success
......................................................................


core: Make VmAnalyzer to treat a migrated Paused VM as success

1. When a VM is unexpectedly reported on a different host in Paused
state, treat this as successful migration, much like the same situation
with VM in Up state. (POSTCOPY paused state is excluded from this case.)

2. When a VM switches from MigratingTo to Paused, treat this as
migration success, the same as switching from MigratingTo to Up.
(POSTCOPY does not appear in this situation, because it comes after
MigratingFrom.)

Change-Id: I189d64cd0280244f8599c6e8d7c0c48c9d949350
Bug-Url: https://bugzilla.redhat.com/1454459
Signed-off-by: Shmuel Melamud <smelamud at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzer.java
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Shmuel Leib Melamud: Verified
  Arik Hadas: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I189d64cd0280244f8599c6e8d7c0c48c9d949350
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list