Change in ovirt-engine[master]: Allow activate hosts that support the engine's version

[ofrenkel at redhat.com]

Omer Frenkel has submitted this change and it was merged.

Change subject: Allow activate hosts that support the engine's version
......................................................................


Allow activate hosts that support the engine's version

HandleVdsVersion prevented activation of any hosts with vdsm version that
was not supported by the engine (SupportedVDSMVersions). This patch
adds validation for the installed Vdsm's supportedENGINEs value. If one of the
two is supported we can allow the activation.

Vdsm minor version might bumped up after new present feature, major
version doesn't change in such case. Under the vdsm capabilities we define the
supported engine's version to allow working with new features even
though the engine's version wasn't changed. If the vdsm version supports the
engine's version its enough to allow the activation.

This check was omitted as part of http://gerrit.ovirt.org/#/c/17719

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1016461
Change-Id: I70b10f9ede37de858df0aa88185c75f2d84fc141
Signed-off-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/HandleVdsVersionCommand.java
1 file changed, 15 insertions(+), 4 deletions(-)

Approvals:
  Omer Frenkel: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I70b10f9ede37de858df0aa88185c75f2d84fc141
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server