Ryan Barry has uploaded a new change for review.

Change subject: Don't hide bridges in the network page
......................................................................

Don't hide bridges in the network page

Show bridges instead of hiding them. They appear as managed, so
there's little risk of users unconfiguring them (bridge interfaces
should not exist without registration in the first place).

Change-Id: I1b543c3be4778482b777635dabe6962823ce40b8
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1097668
Signed-off-by: Ryan Barry <[email protected]>
---
M src/ovirt/node/utils/network.py
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/03/37203/1

diff --git a/src/ovirt/node/utils/network.py b/src/ovirt/node/utils/network.py
old mode 100644
new mode 100755
index 3fc7811..5a9c102
--- a/src/ovirt/node/utils/network.py
+++ b/src/ovirt/node/utils/network.py
@@ -611,7 +611,7 @@
         nics = [NIC(ifname) for ifname
                 in self.relevant_ifnames(filter_vlans=True,
                                          filter_bonds=False,
-                                         filter_bridges=True)]
+                                         filter_bridges=False)]
 
         bridges = [nic for nic in nics if nic.typ == "bridge"]
         vlans = [nic for nic in nics if nic.typ == "vlan"]
@@ -623,7 +623,7 @@
 
         candidates = {}
 
-        for nic in nics + vlans:
+        for nic in nics + vlans + bridges:
             candidate = self.build_nic_model(nic.ifname, filter_slaveless,
                                              filter_configured)
             if candidate:


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1b543c3be4778482b777635dabe6962823ce40b8
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <[email protected]>
_______________________________________________
node-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/node-patches

Reply via email to