Change in ovirt-engine[master]: engine: Block sync of network with QoS when not supported

lvernia at redhat.com lvernia at redhat.com
Sun Jan 19 12:34:30 UTC 2014


Lior Vernia has submitted this change and it was merged.

Change subject: engine: Block sync of network with QoS when not supported
......................................................................


engine: Block sync of network with QoS when not supported

When a QoS entity is attached to network, the network is supposed to
appear as out-of-sync on a host, no matter the cluster compatibility
version of the host. However, when the host's cluster doesn't support
Host Network QoS, it shouldn't be possible to then synchronize the
network (which whould also apply QoS to it).

Change-Id: Ie79c6463676b82f1743fb162fc6c660dd88756f8
Bug-Url: https://bugzilla.redhat.com/1054320
Signed-off-by: Lior Vernia <lvernia at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/SetupNetworksHelper.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/host/SetupNetworksHelperTest.java
2 files changed, 54 insertions(+), 21 deletions(-)

Approvals:
  Lior Vernia: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie79c6463676b82f1743fb162fc6c660dd88756f8
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni 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