Open Stack

On 8/18/20 9:53 AM, Rabi Mishra wrote:
> 
> 
> On Tue, Aug 18, 2020 at 1:07 PM Cédric Jeanneret <cjeanner at redhat.com
> <mailto:cjeanner at redhat.com>> wrote:
> 
>     Hello there!
> 
>     I'm wondering if we could move the "tripleo-ansible-inventory" script
>     from the tripleo-validations repo to tripleo-common.
> 
> 
> TBH, I don't know the history, but it would be better if we remove all
> scripts from tripleo-common and use it just as a utility library (now
> that Mistral is gone). Most of the existing scripts probably have an
> existing command in tripleoclient. We can implement  missing ones
> including "tripleo-ansible-inventory" in python-tripleoclient.

would probably be better to implement it directly in tripleoclient imho.
In any cases, it has nothing to do in tripleo-validations...

I can't connect to launchpad, they are having some auth issue, I can't
create an RFE there :(.

> 
> 
>     The main motivation here is to make things consistent:
>     - that script calls content from tripleo-common, nothing from
>     tripleo-validations.
>     - that script isn't only for the validations, so it makes more sense to
>     install it via tripleo-common
>     - in fact, we should probably push that inventory thing as an `openstack
>     tripleo' sub-command, but that's another story
> 
>     So, is there any opposition to this proposal?
> 
>     Cheers,
> 
>     C.
> 
> 
>     -- 
>     Cédric Jeanneret (He/Him/His)
>     Sr. Software Engineer - OpenStack Platform
>     Deployment Framework TC
>     Red Hat EMEA
>     https://www.redhat.com/
> 
> 
> 
> -- 
> Regards,
> Rabi Mishra
> 

-- 
Cédric Jeanneret (He/Him/His)
Sr. Software Engineer - OpenStack Platform
Deployment Framework TC
Red Hat EMEA
https://www.redhat.com/

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://lists.openstack.org/pipermail/openstack-discuss/attachments/20200818/6026fbcf/attachment.sig>