Change in vdsm[master]: hooks: ovs: move libvirt configuration into a separate module

danken at redhat.com danken at redhat.com
Sun Oct 11 09:06:50 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: hooks: ovs: move libvirt configuration into a separate module
......................................................................


hooks: ovs: move libvirt configuration into a separate module

Libvirt configuration has nothing to do with OVS configuration.

The main reason is to move creation of new libvirt networks to a place
where almost all configuration is completed, so an eventual rollback
doesn't have to do any extra work. This is necessary for the following
'improve ovs rollback' patch.

Change-Id: I775edc776c738822ed955ffb0b7885dae450b667
Signed-off-by: Petr Horáček <phoracek at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/46910
Reviewed-by: Ondřej Svoboda <osvoboda at redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-by: Ido Barkan <ibarkan at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M debian/vdsm-hook-ovs.install
M vdsm.spec.in
M vdsm_hooks/ovs/Makefile.am
M vdsm_hooks/ovs/ovs_before_network_setup.py
A vdsm_hooks/ovs/ovs_before_network_setup_libvirt.py
M vdsm_hooks/ovs/ovs_before_network_setup_ovs.py
6 files changed, 73 insertions(+), 38 deletions(-)

Approvals:
  Ido Barkan: Looks good to me, but someone else must approve
  Ondřej Svoboda: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Petr Horáček: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I775edc776c738822ed955ffb0b7885dae450b667
Gerrit-PatchSet: 9
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: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list