Yaniv Bronhaim 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/1196271 Reviewed-on: http://gerrit.ovirt.org/37751 Reviewed-by: Dan Kenigsberg <[email protected]> Reviewed-on: https://gerrit.ovirt.org/38079 --- 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 https://gerrit.ovirt.org/38079 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8b472305c00e3f11a4f7ede78f84eea589ca616b Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Petr Horáček <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Petr Horáček <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[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
