Bug#848314: [Pkg-libvirt-maintainers] Bug#848314: Further fix for discussion in the scope of smoke-qemu-session

2016-12-19 Thread Christian Ehrhardt
On Mon, Dec 19, 2016 at 9:29 AM, Christian Ehrhardt < christian.ehrha...@canonical.com> wrote: > I'm now rebasing to integrate on top of your last 5 commits of Friday and > check again. > But I assume I'll hit the same. > Qemu is not yet merged to latest yet, so still at 2.6.1 there - which is > o

Bug#848314: [Pkg-libvirt-maintainers] Bug#848314: Further fix for discussion in the scope of smoke-qemu-session

2016-12-19 Thread Christian Ehrhardt
On Fri, Dec 16, 2016 at 11:06 AM, Christian Ehrhardt < christian.ehrha...@canonical.com> wrote: > I'll be throwing that against all architectures and get back to you if it > works for me. > (IIRC you already accepted the d/t/control dependency to qemu-system in my > former bug, so not listed) > H

Bug#848314: [Pkg-libvirt-maintainers] Bug#848314: Further fix for discussion in the scope of smoke-qemu-session

2016-12-16 Thread Christian Ehrhardt
On Fri, Dec 16, 2016 at 10:42 AM, Guido Günther wrote: > You want qemu-system running as non root not qemu-user. > gah - sorry my bad - of course not cross emulation user mode. > > -/usr/bin/kvm > > +/usr/bin/qemu-x86_64 > > Like this but using qemu-system-x86_64. > Yeah as follow on

Bug#848314: [Pkg-libvirt-maintainers] Bug#848314: Further fix for discussion in the scope of smoke-qemu-session

2016-12-16 Thread Guido Günther
On Fri, Dec 16, 2016 at 10:31:58AM +0100, Christian Ehrhardt wrote: > On Fri, Dec 16, 2016 at 10:07 AM, Guido Günther wrote: > > > The fix is to use qemu not kvm in the domain XML in this case. > > > If that is the way to go - to focus on testing qemu:/// - that is right. > The current call to

Bug#848314: [Pkg-libvirt-maintainers] Bug#848314: Further fix for discussion in the scope of smoke-qemu-session

2016-12-16 Thread Christian Ehrhardt
On Fri, Dec 16, 2016 at 10:07 AM, Guido Günther wrote: > The fix is to use qemu not kvm in the domain XML in this case. If that is the way to go - to focus on testing qemu:/// - that is right. The current call to virt-host-validate qemu already has a || true so it is not fatal which is good. W

Bug#848314: [Pkg-libvirt-maintainers] Bug#848314: Further fix for discussion in the scope of smoke-qemu-session

2016-12-16 Thread Guido Günther
On Fri, Dec 16, 2016 at 10:02:23AM +0100, Christian Ehrhardt wrote: > Package: libvirt > Version: 2.5.0-1 > Severity: normal > > This is a continuation of debbug 848247. > But you said you want to close that with next upload but I should come back. > > While the tests are now working fine in loca