Change in ovirt-host-deploy[master]: vdsm: bridge: attempt to rollback if bridge creation failed

[Alon Bar-Lev]

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 at 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 at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>