Hi, thanks for the notice and for the work on the plugin. You plugin looks good to me, we can go with yours. Just one note - we need to provide a support to filter the VMs by the interface. The biggest gap in previous dynamic inventory script was that it picked first interface with IP, but users want to choose which interface to pick. On 10/24/18 10:59 PM, verschueren.bram(a)gmail.com wrote: > Hi, > > Some while ago I started working on an Ansible inventory plugin for ovirt. > Now I just noticed there was a ticket opened by Ondra (#45104) on the github repo and I wonder if anyone is actively working on this, or if you would consider to adopt the plugin I created. > I've added it to my repo at https://github.com/bverschueren/ansible/blob/ovirt_inventory_plugin/lib/a... > > Regards, > Bram > _______________________________________________ > Devel mailing list -- devel(a)ovirt.org > To unsubscribe send an email to devel-leave(a)ovirt.org > Privacy Statement: https://www.ovirt.org/site/privacy-policy/ > oVirt Code of Conduct: https://www.ovirt.org/community/about/community-guidelines/ > List Archives: https://lists.ovirt.org/archives/list/devel@ovirt.org/message/SMRY5FLLNFQ... >

Meanwhile I've added caching too, but I've seen you have a PR open for your own version. Do you prefer to proceed with your version or shall I create a PR too?