[node-patches] Change in ovirt-node[master]: Apply default disable on network details page

fabiand at fedoraproject.org fabiand at fedoraproject.org
Wed Jun 25 14:59:29 UTC 2014


Fabian Deutsch has submitted this change and it was merged.

Change subject: Apply default disable on network details page
......................................................................


Apply default disable on network details page

We should check for the value being "none" in case of users
manually selection the "Disabled" checkbox as well as None in case
they never picked it at all. This way, we can avoid a situation
where users leave auto/dhcp enabled by failing to make a selection
for that field and saving the page.

Change-Id: I406f250e7cd2f3718d063826dcbe48cb1cbd57ef
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1094552
Signed-off-by: Ryan Barry <rbarry at redhat.com>
---
M src/ovirt/node/setup/core/network_page.py
1 file changed, 2 insertions(+), 2 deletions(-)



-- 
To view, visit http://gerrit.ovirt.org/27480
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I406f250e7cd2f3718d063826dcbe48cb1cbd57ef
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the node-patches mailing list