[node-patches] Change in ovirt-node[master]: setup: check bridge bootproto instead of device when renamed
mburns at redhat.com
mburns at redhat.com
Wed Oct 10 19:17:57 UTC 2012
Michael Burns has posted comments on this change.
Change subject: setup: check bridge bootproto instead of device when renamed
......................................................................
Patch Set 1: (1 inline comment)
looks fine...but see inline before pushing
....................................................
File scripts/ovirt-config-setup.py
Line 1064: dev_bridge) = self.nic_dict[dev].split(",", 6)
Line 1065: if dev_bridge == "rhevm" or dev_bridge == "ovirtmgmt":
Line 1066: cmd = ("/files/etc/sysconfig/network-scripts/" + \
Line 1067: "ifcfg-%s/BOOTPROTO" % dev_bridge)
Line 1068: bootproto = augtool_get(cmd)
why only rhevm or ovirtmgmt? shouldn't we read the bridge device for all devices if it's defined?
Line 1069: nic_detail_grid = Grid(6, 10)
Line 1070: nic_detail_grid.setField(Label("Interface: "), 0, 1, anchorLeft=1,
Line 1071: padding=(0, 0, 1, 0))
Line 1072: nic_detail_grid.setField(Label("Protocol: "), 0, 2, anchorLeft=1,
--
To view, visit http://gerrit.ovirt.org/8475
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I710ae355bad1c71ae7f9a3bf35d6e9c741b5c5ed
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Joey Boggs <jboggs at redhat.com>
Gerrit-Reviewer: Michael Burns <mburns at redhat.com>
More information about the node-patches
mailing list