Change in vdsm[master]: ifcfg: stop leaving leftover bridges when removing bonded ne...

danken at redhat.com danken at redhat.com
Mon Mar 3 18:31:26 UTC 2014


Dan Kenigsberg has submitted this change and it was merged.

Change subject: ifcfg: stop leaving leftover bridges when removing bonded networks
......................................................................


ifcfg: stop leaving leftover bridges when removing bonded networks

When having a network bridged non-vlanned bonded network, removing
the network while intending to remove the bond would remove the
network bridge and then create it again.

This patch fixes it by removing the 'BRIDGE' entry from the bond
before ifuppin it up again.

Change-Id: I3de692c7f089d6a9f404c3a2f6f568c12d8a985d
Bug-Url: https://bugzilla.redhat.com/1071398
Signed-off-by: Antoni S. Puimedon <asegurap at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/25203
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Tested-by: Dan Kenigsberg <danken at redhat.com>
---
M tests/functional/networkTests.py
M vdsm/netconf/ifcfg.py
2 files changed, 30 insertions(+), 4 deletions(-)

Approvals:
  Antoni Segura Puimedon: Looks good to me, but someone else must approve
  Dan Kenigsberg: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3de692c7f089d6a9f404c3a2f6f568c12d8a985d
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list