[node-patches] Change in ovirt-node[master]: network: Don't modify tagged interfaces nic names

jboggs at redhat.com jboggs at redhat.com
Tue Aug 7 20:44:33 UTC 2012


Joey Boggs has submitted this change and it was merged.

Change subject: network: Don't modify tagged interfaces nic names
......................................................................


network: Don't modify tagged interfaces nic names

A dict is used internally to map interface names to the corresponding
interface informations.
Previously the VLAN ID of "VLAN interfaces" was removed from a interface
name (e.g. eth0.42 became eth0) which lead to wrong
NIC-informations-mappings, because the informations corresponding to a
VLAN interface (eth0.42) overwrote the informations of the original
interface (eth0). This lead to obvious problems because informations like
e.g. vendor were missing from the vlan interfaces informations.
This patch prevents the name mangling of tagged interfaces, this way the
interface is later ignored when summarizing the informations.

rhbz#837745

Change-Id: If0afd40a7f9c0b541f06722cee50568e746c0336
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M scripts/network.py
1 file changed, 0 insertions(+), 2 deletions(-)

Approvals:
  Joey Boggs: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: If0afd40a7f9c0b541f06722cee50568e746c0336
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Joey Boggs <jboggs at redhat.com>



More information about the node-patches mailing list