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

iheim at redhat.com iheim at redhat.com
Tue Feb 4 10:57:20 UTC 2014


Itamar Heim 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:
  Lior Vernia: Verified
  Moti Asayag: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8a5303222d51459019c1aaaf6754867cd234fe9d
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.4
Gerrit-Owner: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list