Vojtech Szocs has submitted this change and it was merged. Change subject: webadmin: Setup Host Networks dialog fixes (#811605) ...................................................................... webadmin: Setup Host Networks dialog fixes (#811605) https://bugzilla.redhat.com/811605 1. Bonding two NICs (NetworkOperation.BOND_WITH) no longer causes logical network(s) assigned to bonded NICs to disappear from the dialog Before NICs are bonded, logical network(s) assigned to them are first detached (unassigned). After bonding, these networks will be present as "Unassigned Logical Networks" in the right-hand side of the dialog. This behavior (detaching networks from NICs before bonding) is consistent with "add NIC to bond" dialog operation (NetworkOperation.ADD_TO_BOND). 2. Fixed a typo in unassigned network group label 3. Removed some redundant/unnecessary methods Change-Id: Ibd21a9dda5eca8bb954d30be28eec5cfa8cbc203 Signed-off-by: Vojtech Szocs <vszocs@redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/network/NetworkOperation.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostSetupNetworksPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NetworkGroup.java 4 files changed, 8 insertions(+), 18 deletions(-) Approvals: Vojtech Szocs: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/4757 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibd21a9dda5eca8bb954d30be28eec5cfa8cbc203 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs@redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan@redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen@redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik@redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs@redhat.com>