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@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@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl@redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel@redhat.com> Gerrit-Reviewer: Roy Golan <rgolan@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server