<div dir="ltr"><div dir="ltr"><div>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.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 18 Aug 2020 at 11:11, Cédric Jeanneret <<a href="mailto:cjeanner@redhat.com">cjeanner@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
On 8/18/20 9:53 AM, Rabi Mishra wrote:<br>
> <br>
> <br>
> On Tue, Aug 18, 2020 at 1:07 PM Cédric Jeanneret <<a href="mailto:cjeanner@redhat.com" target="_blank">cjeanner@redhat.com</a><br>
> <mailto:<a href="mailto:cjeanner@redhat.com" target="_blank">cjeanner@redhat.com</a>>> wrote:<br>
> <br>
>     Hello there!<br>
> <br>
>     I'm wondering if we could move the "tripleo-ansible-inventory" script<br>
>     from the tripleo-validations repo to tripleo-common.<br>
> <br>
> <br>
> TBH, I don't know the history, but it would be better if we remove all<br>
> scripts from tripleo-common and use it just as a utility library (now<br>
> that Mistral is gone). Most of the existing scripts probably have an<br>
> existing command in tripleoclient. We can implement  missing ones<br>
> including "tripleo-ansible-inventory" in python-tripleoclient.<br>
<br>
would probably be better to implement it directly in tripleoclient imho.<br>
In any cases, it has nothing to do in tripleo-validations...<br></blockquote><div><br></div><div>Moving it to tripleoclient makes sense IMO.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
I can't connect to launchpad, they are having some auth issue, I can't<br>
create an RFE there :(.<br>
<br>
> <br>
> <br>
>     The main motivation here is to make things consistent:<br>
>     - that script calls content from tripleo-common, nothing from<br>
>     tripleo-validations.<br>
>     - that script isn't only for the validations, so it makes more sense to<br>
>     install it via tripleo-common<br>
>     - in fact, we should probably push that inventory thing as an `openstack<br>
>     tripleo' sub-command, but that's another story<br>
> <br>
>     So, is there any opposition to this proposal?<br>
> <br>
>     Cheers,<br>
> <br>
>     C.<br>
> <br>
> <br>
>     -- <br>
>     Cédric Jeanneret (He/Him/His)<br>
>     Sr. Software Engineer - OpenStack Platform<br>
>     Deployment Framework TC<br>
>     Red Hat EMEA<br>
>     <a href="https://www.redhat.com/" rel="noreferrer" target="_blank">https://www.redhat.com/</a><br>
> <br>
> <br>
> <br>
> -- <br>
> Regards,<br>
> Rabi Mishra<br>
> <br>
<br>
-- <br>
Cédric Jeanneret (He/Him/His)<br>
Sr. Software Engineer - OpenStack Platform<br>
Deployment Framework TC<br>
Red Hat EMEA<br>
<a href="https://www.redhat.com/" rel="noreferrer" target="_blank">https://www.redhat.com/</a><br>
<br>
</blockquote></div></div>