Change in ovirt-engine[ovirt-engine-3.6.3]: core: Enable fencing of previous host which HE VM was runnin...

[piotr.kliczewski at gmail.com]

Piotr Kliczewski 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 at 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/BaseDao.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/03_06_2030-add-index-on-origin-in-vm-static.sql
M packaging/dbscripts/vms_sp.sql
9 files changed, 141 insertions(+), 2 deletions(-)

Approvals:
  Martin Peřina: Verified
  Jenkins CI: Passed CI tests
  Oved Ourfali: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iff4368d02d3cbb71dd3cff9c2fc62586fa560a14
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6.3
Gerrit-Owner: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>