Martin Peřina has submitted this change and it was merged. Change subject: core: Enable fencing of previous host which HE VM was running on ...................................................................... core: Enable fencing of previous host which HE VM was running on If host (for example hostA), which hosted engine VM was running on, crashes, then hosted engine VM is restarted on another host (hostB). When engine starts up on hostB, hostA is detected as non responsive, but it's not fenced automatically because fencing is disabled for 5 minutes after engine startup (config value DisableFenceAtStartupInSec). This causes issues: 1. HA VMs executed on hostA are not restarted on different host 2. If hostA was SPM, new SPM cannot be selected This patch enables fencing of hostA even during DisableFenceAtStartupInSec timeout to prevent above issues. Change-Id: Iff4368d02d3cbb71dd3cff9c2fc62586fa560a14 Bug-Url: https://bugzilla.redhat.com/1266099 Signed-off-by: Martin Perina <mperina@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostedengine/PreviousHostedEngineHost.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/pm/VdsNotRespondingTreatmentCommand.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDaoImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmDaoTest.java A packaging/dbscripts/upgrade/04_00_0250-add-index-on-origin-in-vm-static.sql M packaging/dbscripts/vms_sp.sql 8 files changed, 126 insertions(+), 2 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved Arik Hadas: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/53436 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iff4368d02d3cbb71dd3cff9c2fc62586fa560a14 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Eli Mesika <emesika@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina@redhat.com> Gerrit-Reviewer: Martin Sivák <msivak@redhat.com> Gerrit-Reviewer: Moti Asayag <masayag@redhat.com> Gerrit-Reviewer: Roy Golan <rgolan@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>