On Tue, Aug 18, 2020 at 10:08 AM Cédric Jeanneret <cjeanner@redhat.com> wrote:


On 8/18/20 9:53 AM, Rabi Mishra wrote:
>
>
> On Tue, Aug 18, 2020 at 1:07 PM Cédric Jeanneret <cjeanner@redhat.com
> <mailto:cjeanner@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...

+1 with that, it will probably be better to move everything in tripleoclient.
 

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/