[Libguestfs] [PATCH 0/7] RFC: switch v2v to ocaml-libvirt

[Richard W.M. Jones]

On Thu, Aug 30, 2018 at 07:01:00PM +0200, Pino Toscano wrote:
> Hi,
> 
> this is a mostly done attempt to switch to ocaml-libvirt, embedding the
> latest version of it from git. This way, it is possible to improve the
> way v2v connects to libvirt for both input, and output modules, and
> interacts with libvirt (e.g. no more virsh calls needed in virt-v2v).
> 
> As side effect, virt-v2v now requires libvirt, as keeping it optional
> would create too much burden.
> 
> I could not test all the libvirt input modes (like VDDK, and Xen), but
> VMware and libvirtxml work fine as before.

I've not looked into this in detail.

I think because of the new dependency ocaml-libvirt (and indeed mainly
because of the abysmal state of that library for many years) we should
delay this change to after 1.40.

However I have no objection in principle.

Rich.

> 
> Pino Toscano (7):
>   v2v: require libvirt
>   common: Bundle the ocaml-libvirt library for use by virt-v2v
>   v2v: switch to ocaml-libvirt
>   v2v: -o libvirt: use a Lazy for the connection
>   v2v: -o libvirt: switch away from virsh
>   v2v: test-harness: stop using the external ocaml-libvirt
>   build: stop looking for ocaml-libvirt
> 
>  .gitignore                            |    2 +
>  Makefile.am                           |    5 +-
>  common/mllibvirt/Makefile.am          |  102 ++
>  common/mllibvirt/generator.pl         |  890 ++++++++++++++
>  common/mllibvirt/libvirt.README       |    9 +
>  common/mllibvirt/libvirt.ml           | 1624 +++++++++++++++++++++++++
>  common/mllibvirt/libvirt.mli          | 1537 +++++++++++++++++++++++
>  common/mllibvirt/libvirt_c_epilogue.c |  420 +++++++
>  common/mllibvirt/libvirt_c_oneoffs.c  | 1550 +++++++++++++++++++++++
>  common/mllibvirt/libvirt_c_prologue.c |  129 ++
>  configure.ac                          |    1 +
>  docs/C_SOURCE_FILES                   |    1 -
>  m4/guestfs-ocaml.m4                   |    4 -
>  po/POTFILES                           |    1 -
>  v2v/Makefile.am                       |   26 +-
>  v2v/copy_to_local.ml                  |    7 +-
>  v2v/dummy.c                           |    2 +
>  v2v/input_libvirt.ml                  |   20 +-
>  v2v/input_libvirt_other.ml            |   27 +-
>  v2v/input_libvirt_other.mli           |    5 +-
>  v2v/input_libvirt_vcenter_https.ml    |   13 +-
>  v2v/input_libvirt_vcenter_https.mli   |    2 +-
>  v2v/input_libvirt_vddk.ml             |   21 +-
>  v2v/input_libvirt_vddk.mli            |    4 +-
>  v2v/input_libvirt_xen_ssh.ml          |   13 +-
>  v2v/input_libvirt_xen_ssh.mli         |    2 +-
>  v2v/input_libvirtxml.ml               |    3 +-
>  v2v/libvirt_utils-c.c                 |  539 --------
>  v2v/libvirt_utils.ml                  |   95 +-
>  v2v/libvirt_utils.mli                 |   51 +-
>  v2v/output_libvirt.ml                 |   56 +-
>  v2v/parse_libvirt_xml.ml              |   14 +-
>  v2v/parse_libvirt_xml.mli             |   11 +-
>  v2v/test-harness/Makefile.am          |    5 +-
>  v2v/v2v.ml                            |   12 +-
>  35 files changed, 6500 insertions(+), 703 deletions(-)
>  create mode 100644 common/mllibvirt/Makefile.am
>  create mode 100755 common/mllibvirt/generator.pl
>  create mode 100644 common/mllibvirt/libvirt.README
>  create mode 100644 common/mllibvirt/libvirt.ml
>  create mode 100644 common/mllibvirt/libvirt.mli
>  create mode 100644 common/mllibvirt/libvirt_c_epilogue.c
>  create mode 100644 common/mllibvirt/libvirt_c_oneoffs.c
>  create mode 100644 common/mllibvirt/libvirt_c_prologue.c
>  create mode 100644 v2v/dummy.c
>  delete mode 100644 v2v/libvirt_utils-c.c
> 
> -- 
> 2.17.1
> 
> _______________________________________________
> Libguestfs mailing list
> Libguestfs at 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-df lists disk usage of guests without needing to install any
software inside the virtual machine.  Supports Linux and Windows.
http://people.redhat.com/~rjones/virt-df/