Change in ovirt-engine[master]: Allow activate hosts that support the engine's version
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(a)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(a)redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server