Change in ovirt-engine[ovirt-engine-3.6]: engine, webadmin: do not invoke HostSetupNetworksCommand wher...
Tal Nisan has submitted this change and it was merged.
Change subject: engine,webadmin: do not invoke HostSetupNetworksCommand where no changes
were made
......................................................................
engine,webadmin: do not invoke HostSetupNetworksCommand where no changes were made
• discard from HostSetupNetworksCommandParameters all bonds and
network attachments, which are sent to be updated, but they're
equal to already existing Bonds resp. NetworkAttachment in the system.
• added method to HostSetupNetworksCommandParameters to figure out if
there's any change to be done in them, and this method is used both on UI and
bll to figure out, whether changes needs to be done to the system.
Change-Id: Icc4c17171ed37cea79ea80aa65b7f2a056ca407c
Bug-Url: https://bugzilla.redhat.com/1254320
Signed-off-by: Martin Mucha <mmucha(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/GetHostBondsByHostIdQuery.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/HostSetupNetworksCommand.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/HostSetupNetworksParameters.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostSetupNetworksModel.java
4 files changed, 145 insertions(+), 35 deletions(-)
Approvals:
Martin Mucha: Verified
Alona Kaplan: Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/49435
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Icc4c17171ed37cea79ea80aa65b7f2a056ca407c
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>