> 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: 701...@bugs.debian.org >> > >> Reported-by: Ernesto Domato <edo...@gmail.com> >> > >> Suggested-by: Lukasz Szotek <szot...@gmail.com> >> > >> Signed-off-by: Ben Pfaff <b...@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 <yamam...@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). _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
