Change in ovirt-engine[master]: engine: Check VDSM supported clusters to deserialize QoS

masayag at redhat.com masayag at redhat.com
Tue Feb 4 09:16:33 UTC 2014


Moti Asayag has submitted this change and it was merged.

Change subject: engine: Check VDSM supported clusters to deserialize QoS
......................................................................


engine: Check VDSM supported clusters to deserialize QoS

It is not in fact correct check the cluster's compatibility version to
see if Host Network QoS is supported, since GetVdsCapabilities should
work even if the VDSM is too old for the cluster's compatibility
version, and in case there's QoS configured on a host we'd want it to
be reported even if it resides in an older cluster.

Change-Id: I8a5303222d51459019c1aaaf6754867cd234fe9d
Bug-Url: https://bugzilla.redhat.com/1056155
Signed-off-by: Lior Vernia <lvernia at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java
1 file changed, 4 insertions(+), 2 deletions(-)

Approvals:
  Moti Asayag: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8a5303222d51459019c1aaaf6754867cd234fe9d
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list