[libvirt] [jenkins-ci PATCH v2 04/12] lcitool: Add inventory handling

Andrea Bolognani abologna at redhat.com
Tue Jul 17 12:51:47 UTC 2018


On Tue, 2018-07-17 at 11:57 +0200, Katerina Koukiou wrote:
> On Thu, Jul 12, 2018 at 05:19:21PM +0200, Andrea Bolognani wrote:
> > +import yaml
> 
> Since you have to install yaml to actually use it, I suggest at this
> point to create a requirements.txt file for this script, so that users
> don't have to go over the code to check the dependencies.

Neat idea.

If that's okay with you, I'd rather work on that as a follow-up
enhancement though, because I need to do some reading on it and
I'd rather not leave this series lingering for too long unless
it's really necessary.

-- 
Andrea Bolognani / Red Hat / Virtualization




More information about the libvir-list mailing list