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

[mperina at redhat.com]

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 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/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 at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>