[ovirt-ansible] inventory plugin

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/ansi... Regards, Bram

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@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/ansi...
Regards, Bram _______________________________________________ Devel mailing list -- devel@ovirt.org To unsubscribe send an email to devel-leave@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/SMRY5FLLNFQ27G...

Hi, I presume you mean selecting the interface which provides the ansible_host ip ? I've updated the plugin with such a selector. Also I noticed that the reported_devices_service().list() returns the interfaces in reversed alphabetical order so I had to re-reverse them to pick eth0 instead of ethX if no preferred interface is given. Regards, Bram

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?

Please create PR, I will close mine. On 11/15/18 9:54 AM, Bram Verschueren wrote:
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? _______________________________________________ Devel mailing list -- devel@ovirt.org To unsubscribe send an email to devel-leave@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/MOJPUF5P4BV5TS...
participants (3)
-
Bram Verschueren
-
Ondra Machacek
-
verschueren.bram@gmail.com