Re: [libvirt] [jenkins-ci PATCH v2 07/12] lcitool: Implement the 'update' action
On Tue, Jul 17, 2018 at 04:45:39PM +0200, Andrea Bolognani wrote:
On Tue, 2018-07-17 at 14:57 +0200, Katerina Koukiou wrote:
> On Thu, Jul 12, 2018 at 05:19:24PM +0200, Andrea Bolognani wrote:
> > + cmd = [ "ansible-playbook" ]
>
> IMO this requirement should be as well in python requirements file.
> In this way you can also specify which ansible version is expected for the
> site.yaml playbook to run.
I had no idea you could require commands as well as Python modules.
Clearly shows that I need to read up on the stuff :)
Ansible is available from pypi https://pypi.org/project/ansible/
Well I had in mind only pip requirements just put together in a
requirements file so that you can install them with `pip install -r
requirements.txt`.
Then you can easily test different ansible versions for example.
Anyway, of course this can be addressed in the future.
--
Andrea Bolognani / Red Hat / Virtualization
Attachments:
- signature.asc (application/pgp-signature — 488 bytes)