Dan Kenigsberg has submitted this change and it was merged. Change subject: network: api: allow removal of bond with no slaves ......................................................................
network: api: allow removal of bond with no slaves If we try to remove a bond with no slaves attached, VDSM explodes and doesn't allow us to repair the network. This patch moves checking of slaves up the stack, only to 'edit' and 'add' bond's functions. 'remove' bond function is able to objectivize slaveless bond and allows us to remove it. Change-Id: I8b472305c00e3f11a4f7ede78f84eea589ca616b Signed-off-by: Petr Horáček <[email protected]> Bug-Url: https://bugzilla.redhat.com/1183728 Reviewed-on: http://gerrit.ovirt.org/37751 Reviewed-by: Dan Kenigsberg <[email protected]> --- M tests/functional/networkTests.py M vdsm/network/api.py M vdsm/network/models.py 3 files changed, 29 insertions(+), 4 deletions(-) Approvals: Petr Horáček: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37751 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8b472305c00e3f11a4f7ede78f84eea589ca616b Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Petr Horáček <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Ondřej Svoboda <[email protected]> Gerrit-Reviewer: Petr Horáček <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
