Change in ovirt-engine[master]: core: Cannot confirm Host has been rebooted when..

emesika at redhat.com emesika at redhat.com
Mon Mar 18 10:42:49 UTC 2013


Eli Mesika has submitted this change and it was merged.

Change subject: core: Cannot confirm Host has been rebooted when..
......................................................................


core: Cannot confirm Host has been rebooted when..

Cannot confirm Host has been rebooted when having a single host in the system.

This patch removes the SPM Host mark in the DC in case that Host is
non-responsive and is the only Host in the DC.

This enables to put the Host in Maintenance and remove it if needed.

Two scenarios tested
1) Making the Host responsive again => Host is SPM back
2) Adding another Host => New Host is SPM

This patch is not handling the case in which the problematic Host has
running VMs, only the scenario reported in the BZ was handled

Change-Id: I8264d91d6dc5f33e4684d3bef86624b6f424a64b
Signed-off-by: Eli Mesika <emesika at redhat.com>
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=837539
Signed-off-by: Eli Mesika <emesika at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/FenceVdsManualyCommand.java
1 file changed, 10 insertions(+), 2 deletions(-)

Approvals:
  Eli Mesika: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/13045
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8264d91d6dc5f33e4684d3bef86624b6f424a64b
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list