Change in ovirt-engine[ovirt-engine-3.6.0]: core: spmStart failure when clusterlock inquire isn't supported

[tnisan at redhat.com]

Tal Nisan has submitted this change and it was merged.

Change subject: core: spmStart failure when clusterlock inquire isn't supported
......................................................................


core: spmStart failure when clusterlock inquire isn't supported

As part of the SPM selection flow in the engine we run getSpmStatus()
on the candidate host to get the current spm status (if possible) and
we act according to the result.

After an "offline" (when the dc is down) DC upgrade from DC that uses
V1 for it's domain (3.0 for example) to a version that uses the
StoragePoolMemoryBackend we might fail when attempting to get the current
spm status, that's because StoragePoolMemoryBackend relies on the fact
that the used clusterlock supports inquiring while it might not.

This patch ignores the failure of getSpmStatus() if the failure is
InquireNotSupportedError and attempts to start the spm, if the cluster
lock is acquired we'll fail on the spmStart flow.

Change-Id: I996fc4454c210dd278dafa9d94a13c4981162f5b
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1242092
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/EngineError.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsProxyData.java
2 files changed, 119 insertions(+), 73 deletions(-)

Approvals:
  Tal Nisan: Verified; Passed CI tests
  Allon Mureinik: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I996fc4454c210dd278dafa9d94a13c4981162f5b
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6.0
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org