> 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: [email protected] >> > >> Reported-by: Ernesto Domato <[email protected]> >> > >> Suggested-by: Lukasz Szotek <[email protected]> >> > >> Signed-off-by: Ben Pfaff <[email protected]> >> > > >> > > 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 <[email protected]> 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). _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
