On Wed, 7 Nov 2018 15:31:56 +0000
"Richard W.M. Jones" <rjones(a)redhat.com> wrote:
 On Wed, Nov 07, 2018 at 12:53:17PM +0100, Tomáš Golembiovský wrote:
 > changes in v3:
 > - fix call to install_local
 > 
 > changes in v2:
 > - moved copy_drivers above copy_files
 > - renamed copy_files to copy_from_virtio_win
 > - renamed install to install_local
 > - use rpm instead of yum
 > 
 > This installs packages with QEMU Guest Agent when converting Linux machine. The
 > packages should be available on guest tools ISO. The patches work "as-is"
but
 > probably deserve some more attention:
 > 
 > - it is "abusing" Winows_virtio code but renaming/refactoring everything
to
 >   remove "windows" from the name and use "guest tools" seems
like a lot of
 >   unnecesary work  
 
 I didn't mean the comment to mean it was abusing Windows_virtio.  It
 was just a surprise since at the moment Convert_windows is the only
 consumer of the Windows_virtio code.  However the change is still
 correct so this doesn't matter, and maybe we can rename the module in
 future. 
This was in cover letter before your comment. :)
 
 > - support for Debian is missing
 >   I don't know how to install the package only when all it's dependencies
are
 >   already installed. dpkg cannot be used to check that (simulate the install).
 >   And attempting to install the package will leave it half-installed (dpkg
 >   cannot roll-back).  
 
 Will we actually have Debian packages on the ISO? 
Yes.
 
 BTW do we actually have an example of the new ISO or virtio-win.rpm I
 can test against? 
I am not sure if we have ISO for oVirt already. Let me check.
    Tomas
 
 Rich.
 
 > Tomáš Golembiovský (3):
 >   v2v: refactor copy_drivers() in Windows_virtio
 >   v2v: linux: install packages
 >   v2v: linux: install QEMU-GA (RHBZ#1619665)
 > 
 >  v2v/convert_linux.ml   |  2 ++
 >  v2v/linux.ml           | 14 ++++++++
 >  v2v/linux.mli          |  3 ++
 >  v2v/windows_virtio.ml  | 78 +++++++++++++++++++++++++++++++++---------
 >  v2v/windows_virtio.mli |  4 +++
 >  5 files changed, 84 insertions(+), 17 deletions(-)
 > 
 > -- 
 > 2.19.0
 > 
 > _______________________________________________
 > Libguestfs mailing list
 > Libguestfs(a)redhat.com
 > 
https://www.redhat.com/mailman/listinfo/libguestfs  
 
 -- 
 Richard Jones, Virtualization Group, Red Hat 
http://people.redhat.com/~rjones
 Read my programming and virtualization blog: 
http://rwmj.wordpress.com
 virt-top is 'top' for virtual machines.  Tiny program with many
 powerful monitoring features, net stats, disk stats, logging, etc.
 
http://people.redhat.com/~rjones/virt-top 
-- 
Tomáš Golembiovský <tgolembi(a)redhat.com>