Change in vdsm[master]: net: legacy: split bonds handling to removal and setup

phoracek at redhat.com phoracek at redhat.com
Thu Mar 10 19:04:10 UTC 2016


Petr Horáček has posted comments on this change.

Change subject: net: legacy: split bonds handling to removal and setup
......................................................................


Patch Set 2:

@Edy, we have to detach all slaves and then add them. Otherwise we could stuck in the situation when we want to attach a slave which was not yet removed from another switch.

 for switch in switches:
      switch.remove_bonds()
 for switch in switches:
      switch.setup_bonds()

-- 
To view, visit https://gerrit.ovirt.org/54352
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib53d04c89a72d514ffcea93981b6782f6eb4a14d
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: No


More information about the vdsm-patches mailing list