[ovs-dev] Bug#701760: Bug#701760: [PATCH] debian: Stop openvswitch after VMs managed by libvirt.

[YAMAMOTO Takashi]

> On Tue, May 20, 2014 at 07:24:35AM -0700, Ben Pfaff wrote:
>> On Tue, May 20, 2014 at 10:29:11AM +0900, YAMAMOTO Takashi wrote:
>> > > On Mon, Mar 31, 2014 at 08:37:48AM -0700, Ben Pfaff wrote:
>> > >> When openvswitch stops before libvirt shuts down VMs, it makes it hard for
>> > >> libvirt to remove virtual network interfaces (ovs-vsctl cannot access the
>> > >> database socket, which has been removed).  This commit should ensure that
>> > >> the VMs get shut down before openvswitch.
>> > >> 
>> > >> CC: 701760 at bugs.debian.org
>> > >> Reported-by: Ernesto Domato <edomat at gmail.com>
>> > >> Suggested-by: Lukasz Szotek <szotman at gmail.com>
>> > >> Signed-off-by: Ben Pfaff <blp at nicira.com>
>> > > 
>> > > This still needs a review (from ovs-dev).
>> > 
>> > is this still useful?
>> > my installation of ubuntu 13.10 doesn't seem to have libvirt-guests.
>> 
>> Ansis told me the same thing.  It seems package names have changed from
>> the version of libvirt that the reporter uses, to the version of libvirt
>> that is currently in the Debian archive.
> 
> Actually, looking at the latest version of libvirt in the Debian
> archive, version 1.2.4-3, I see that it has /etc/init.d/libvirt-guests,
> same as previous versions.  Ansis and Yamamoto-san, are you aware that
> the dependencies in these init.d files are the names of init.d scripts,
> not the names of packages?  So I think that this change will have the
> desired effect, considering Ansis's comment below as a separate topic:

mine seems older than yours.

ubu5% dpkg -l libvirt-bin
Desired=Unknown/Install/Remove/Purge/Hold
| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend
|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)
||/ Name           Version      Architecture Description
+++-==============-============-============-=================================
ii  libvirt-bin    1.1.1-0ubunt amd64        programs for the libvirt library
ubu5% ls /etc/init.d|grep libvirt
libvirt-bin*
ubu5% grep libvirt-guests /etc/init.d/*
ubu5% 

if the latest version has libvirt-guests, the change looks ok to me.
Acked-by: YAMAMOTO Takashi <yamamoto at valinux.co.jp>

YAMAMOTO Takashi

> 
>> But Ansis also brought up the point that it might make more sense for
>> libvirt to add this dependency in the reverse direction.  I will consult
>> with the libvirt team (probably by reassigning this bug to libvirt with
>> a query).