Moti Asayag has submitted this change and it was merged. Change subject: core: after HostNetworkQos is removed, sync influenced networks. ...................................................................... core: after HostNetworkQos is removed, sync influenced networks. Already existing in UpdateHostNetworkQosCommand was refactored a little extracted into common ancestor and used also in RemoveHostNetworkQosCommand so that removal of HostNetworkQos also sync influenced networks, so that they aren't left in out of sync state. Change-Id: Ifa8111a59d3c93cb0256727bde620d2d2928338f Bug-Url: https://bugzilla.redhat.com/1279771 Signed-off-by: Martin Mucha <mmucha(a)redhat.com&gt; --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/qos/PersistentHostSetupNetworksParametersFactory.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/qos/QosCommandBase.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/qos/RefreshNetworksParametersFactory.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/qos/RemoveHostNetworkQosCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/qos/UpdateHostNetworkQosCommand.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/qos/PersistentHostSetupNetworksParametersFactoryTest.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/qos/RefreshNetworksParametersFactoryTest.java 7 files changed, 393 insertions(+), 125 deletions(-) Approvals: Martin Mucha: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved Yevgeny Zaspitsky: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/59310 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifa8111a59d3c93cb0256727bde620d2d2928338f Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com&gt; Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com&gt; Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com&gt; Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com&gt; Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com&gt; Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com&gt; Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;