<div dir="auto">Hi sadi,</div><div dir="auto"><br></div><div dir="auto">Thanks for this feedback!</div><div dir="auto">I’ll wait for this patch to be merged then, no biggies as it’s currently greetings season’s so no rush xD</div><div dir="auto"><br></div><div dir="auto">I’ll probably have few patches after that especially around unifying options (filtering especially) on few modules.</div><div dir="auto"><br></div><div dir="auto">Thanks for the answer!</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le lun. 26 déc. 2022 à 21:06, Sagi Shnaidman <<a href="mailto:sshnaidm@redhat.com">sshnaidm@redhat.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)"><div dir="ltr"><div>Hi,  Gael,</div><div><br></div><div>Thanks for your contribution! Currently the tox-2.12 CI job always fails, it's because of tox version 4 changes. I add a workaround in the patch <a href="https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868607" target="_blank">https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868607</a> <br></div><div>When it (or other solution) is merged, you're good to go with your patch. Sorry for the inconvenience .</div><div><br></div><div>Thanks<br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Dec 24, 2022 at 4:22 PM Gaël THEROND <<a href="mailto:gael.therond@bitswalk.com" target="_blank">gael.therond@bitswalk.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">Hi ansible collections openstack team!<div><br></div><div dir="auto">I finally had time to list all my issues met with the project, created few bug reports and even contributed to a patch today (minor, mainly copy/paste) however I’ve few questions regarding the CI process!</div><div dir="auto"><br></div><div dir="auto">Overall, what’s the rule with the CI code testing?</div><div dir="auto">I’ve read the contributing guide and had an eye on previous patches to see how it’s used but I’m having a hard time to find a real unified method. For instance, it seems that some module miss CI tasks (such as compute_service_info) or did I missed something?</div><div dir="auto"><br></div><div dir="auto">Thanks a lot for all the good job!</div>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr"><div dir="ltr"><div>Best regards<br></div>Sagi Shnaidman<br></div></div>
</blockquote></div></div>