Change in ovirt-engine[master]: webadmin: Setup Network- does not show the correct boot prot...

gchaplik at redhat.com gchaplik at redhat.com
Thu Dec 6 13:58:23 UTC 2012


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 at 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 at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>



More information about the Engine-commits mailing list