Gilad Chaplik has submitted this change and it was merged.
Change subject: webadmin: Setup Network- does not show the correct boot protocol.
......................................................................
webadmin: Setup Network- does not show the correct boot protocol.
The fix in commit 30f20efd648a1329c996f5c12f31741038bdc0be caused another
bug.
The attach method is used not only for manually attaching but also in the
init dialog process.
That's why the boot protocol was always set to the default values ("dhcp"
for management and "none" for the others) and never showed the real values.
The boot protocol should be reseted to the default values just if the
current boot protocol is null.
Change-Id: I40fe51b46f94bdfa27270c9e5dcd94074b881c03
Bug-Url:https://bugzilla.redhat.com/875724
Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com>
---
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/network/LogicalNetworkModel.java
1 file changed, 12 insertions(+), 9 deletions(-)
Approvals:
Gilad Chaplik: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/9752
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I40fe51b46f94bdfa27270c9e5dcd94074b881c03
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>