IIRC it was in tripleo-validations since that was the first and only user of the script at the time. When it got into use by other flows it just never got moved.

On Tue, 18 Aug 2020 at 11:11, 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...

Moving it to tripleoclient makes sense IMO.


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/