<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Apr 4, 2022 at 9:27 AM Jiri Podivin <<a href="mailto:jpodivin@redhat.com">jpodivin@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"><div dir="ltr"><div>Full disclosure: I have only surface level understanding of how ansible galaxy actually works on the inside.</div><div>My exposure to it is rather limited and it's possible that all of my concerns have perfectly valid responses I'm not aware of.</div><div>Furthermore, I do believe that we could utilize ansible galaxy a bit more than we do.</div><div><br></div><div>That being said, I do think that we should be cautious when changing the way we package and deliver.</div><div>Even if everything works out we are possibly setting ourselves up for a whole new set of possible problems we are unfamiliar with.</div><div>Whether that is an acceptable risk or not is a question for a different avenue however. <br></div></div><br></blockquote><div><br></div><div>In this particular case, we can get away with installing the ansible galaxy collections because we have 'nested' ansible so something like zuul (ansible) calling bash (tripleo-quickstart) calling ansible.  There are other cases (zuul/ansible 'native', not nested) where we have to install such dependencies as python utilities because of the security concerns around allowing collections to be installed on the ansible controller (e.g. see <a href="http://lists.zuul-ci.org/pipermail/zuul-discuss/2021-November/001752.html">http://lists.zuul-ci.org/pipermail/zuul-discuss/2021-November/001752.html</a>).<br></div><div><br></div><div>In this case, we can do the installation of the required ansible bits during the middle "bash" part of the workflow (as you can see in  <a href="https://review.opendev.org/c/openstack/tripleo-quickstart/+/836104" target="_blank">https://review.opendev.org/c/openstack/tripleo-quickstart/+/836104</a>).  There are other cases where we can't (yet?)</div><div><br></div><div>regards, marios</div><div><br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Apr 3, 2022 at 7:10 PM Dmitriy Rabotyagov <<a href="mailto:noonedeadpunk@gmail.com" target="_blank">noonedeadpunk@gmail.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"><div dir="auto">Hey there!<div dir="auto"><br></div><div dir="auto">I have quick question - do you think it's valid approach to install Ansible roles as python packages?</div><div dir="auto">This smells sooooo fishy since ansible-galaxy is a thing along with requirements.yml...</div><div dir="auto"><br></div><div dir="auto">So actual question is - do you have any plans on changing this approach to more Ansible way anytime soon?</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">пт, 1 апр. 2022 г., 8:19 Marios Andreou <<a href="mailto:marios@redhat.com" target="_blank">marios@redhat.com</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>On Fri, Apr 1, 2022 at 12:14 AM Ronelle Landy <<a href="mailto:rlandy@redhat.com" rel="noreferrer" target="_blank">rlandy@redhat.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hello All,<br><div><br></div><div>We have a check/gate blocker on all TripleO quickstart-based jobs, as described in:</div><div><br></div><div><a href="https://bugs.launchpad.net/tripleo/+bug/1967430" rel="noreferrer" target="_blank">https://bugs.launchpad.net/tripleo/+bug/1967430</a><br></div><div><br></div><div>[1] commit to openstack-ansible-os_tempest removed setup.py and is causing failings in all quickstart jobs.</div><div><br></div><div>A revert was proposed but will not be workable - we are waiting on another fix.</div><div><br></div><div>Please hold rechecks until this is resolved.</div><div><br></div><div>[1] <a href="https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/835969" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/835969</a></div><div><br></div></div></blockquote><div><br></div><div><br></div><div>Unfortunately looks like the core group on that repo is empty [1]. I added some folks into CC here that merged the original patch. Folks can you please help us merge the fix at <a href="https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/836091" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/836091</a> </div><div><br></div><div>TripleO gate is blocked until we merge ansible-role-python_venv_build/+/836091</div><div><br></div><div><br></div><div>please help :D</div><div><br></div><div><br></div><div>[1] <a href="https://review.opendev.org/admin/groups/3474fc86368161e5288be01295041a089a1060b3,members" rel="noreferrer" target="_blank">https://review.opendev.org/admin/groups/3474fc86368161e5288be01295041a089a1060b3,members</a></div><div><br></div><div><br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div></div><div>Thank you!</div></div>
</blockquote></div></div>
</blockquote></div>
</blockquote></div>
</blockquote></div></div>