Alon Bar-Lev has submitted this change and it was merged.
Change subject: vdsm: bridge: attempt to rollback if bridge creation failed
......................................................................
vdsm: bridge: attempt to rollback if bridge creation failed
Currently vdsm-bootstrap, host-deploy, vdsm-reg do not rollback network
configuration if addNetwork fails.
The sequence should be:
- addNetwork
- vdsm-restore-net-config -> rollback
- vdsm-store-net-config -> commit
Supporting non-node setup is not important to 3.3 as only legacy nodes
still use the ovirt-host-deploy bridge plugin.
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=979570
Change-Id: Id9e6b0d076f3a16f036e149258454da395569797
Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com>
---
M ChangeLog
M src/plugins/ovirt-host-deploy/vdsm/bridge.py
2 files changed, 47 insertions(+), 14 deletions(-)
Approvals:
Alon Bar-Lev: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/17124
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id9e6b0d076f3a16f036e149258454da395569797
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-host-deploy
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>