Re: [libvirt] [jenkins-ci PATCH v2 10/12] lcitool: Add projects information handling

2018-07-18 Thread Andrea Bolognani
On Tue, 2018-07-17 at 19:55 +0200, Katerina Koukiou wrote: > I meant only having a variable for the playbook location. > > playbook_path = os.getenv('PLAYBOOK_PATH', './')) > > And then use os.path.join for all the others relative paths to the > playbook path. > So that you can actually run the l

Re: [libvirt] [jenkins-ci PATCH v2 10/12] lcitool: Add projects information handling

2018-07-17 Thread Katerina Koukiou
On Tue, Jul 17, 2018 at 04:44:24PM +0200, Andrea Bolognani wrote: > On Tue, 2018-07-17 at 15:04 +0200, Katerina Koukiou wrote: > > On Thu, Jul 12, 2018 at 05:19:27PM +0200, Andrea Bolognani wrote: > > > +class Projects: > > > + > > > +def __init__(self): > > > +try: > > > +w

Re: [libvirt] [jenkins-ci PATCH v2 10/12] lcitool: Add projects information handling

2018-07-17 Thread Andrea Bolognani
On Tue, 2018-07-17 at 15:04 +0200, Katerina Koukiou wrote: > On Thu, Jul 12, 2018 at 05:19:27PM +0200, Andrea Bolognani wrote: > > +class Projects: > > + > > +def __init__(self): > > +try: > > +with open("./vars/mappings.yml", "r") as f: > > There is clear information where

Re: [libvirt] [jenkins-ci PATCH v2 10/12] lcitool: Add projects information handling

2018-07-17 Thread Katerina Koukiou
On Thu, Jul 12, 2018 at 05:19:27PM +0200, Andrea Bolognani wrote: > The original tool's limited scope meant loadins this > information was not needed, but we're going to start > making use of it pretty soon. > > Signed-off-by: Andrea Bolognani > --- > guests/lcitool | 47

[libvirt] [jenkins-ci PATCH v2 10/12] lcitool: Add projects information handling

2018-07-12 Thread Andrea Bolognani
The original tool's limited scope meant loadins this information was not needed, but we're going to start making use of it pretty soon. Signed-off-by: Andrea Bolognani --- guests/lcitool | 47 +++ 1 file changed, 47 insertions(+) diff --git a/guests/l