Liran Zelkha has posted comments on this change.

Change subject: core: Host's Compatibility Version doesn't match the Cluster's 
Compatibility Version
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.ovirt.org/#/c/26061/2/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/HandleVdsVersionCommand.java
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/HandleVdsVersionCommand.java:

Line 59: 
Line 60:         // If vdsm doesn't support the engine's version (engine's 
version is not included
Line 61:         // vdsm supprtedEngineVersions list) we move on and check if 
engine
Line 62:         // and cluster supports the specific vdsm version. which is 
sufficient
Line 63:         if (!isEngineSupportedByVdsm) {
> Please check also if SupportedVDSMVersions is still used or we should clean
Done
Line 64:             
reportNonOperationReason(NonOperationalReason.VERSION_INCOMPATIBLE_WITH_CLUSTER,
Line 65:                     partialVdcVersion.toString(),
Line 66:                     vdsmVersion.toString());
Line 67:         }


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1a899462ce3d74f3b1570a23c10ce38aeae1a18e
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liran Zelkha <[email protected]>
Gerrit-Reviewer: Barak Azulay <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Liran Zelkha <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to