<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Trying to look  through the logs of the last merged patch set [1] but nothing there. Is there any other places that I can look into?</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">[1] <a href="https://review.openstack.org/#/c/616056/">https://review.openstack.org/#/c/616056/</a><br></div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Thanks,</div></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Dec 12, 2018 at 9:47 AM Trinh Nguyen <<a href="mailto:dangtrinhnt@gmail.com">dangtrinhnt@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="ltr"><div class="gmail_default" style="font-size:small">Thank Jeremy. I will look into bindep.</div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Dec 12, 2018 at 12:04 AM Jeremy Stanley <<a href="mailto:fungi@yuggoth.org" target="_blank">fungi@yuggoth.org</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">On 2018-12-11 15:21:47 +0900 (+0900), Trinh Nguyen wrote:<br>
> I'm trying to figure out how Zuul processes the test-setup.sh [1]<br>
> file to adopt new dependencies.<br>
[...]<br>
<br>
Looking at what you've put in there for your JRE packages, those<br>
would be better handled in a bindep.txt file in your repository.<br>
Running tools/test-setup.sh in jobs is really to catch other sorts<br>
of setup steps like precreating a database or formatting and<br>
mounting a particular filesystem your tests are going to rely on.<br>
For system package dependencies of your jobs, we have a declarative<br>
mechanism already which doesn't require complex conditional<br>
handling: we install whatever bindep says is missing.<br>
<br>
I see you're also leveraging tools/test-setup.sh to obtain packages<br>
of elasticsearch which are not available in the distributions on<br>
which your jobs run, which I guess is a suitable workaround though<br>
it seems odd to test on platforms which don't package the software<br>
on which your project relies.<br>
-- <br>
Jeremy Stanley<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail-m_8205446671626807959gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><b style="font-size:small;color:rgb(51,51,51)">Trinh Nguyen</b><br></div><div><u style="font-size:12.8px;color:rgb(0,0,0)"><a href="https://www.edlab.xyz" target="_blank">www.edlab.xyz</a></u><br></div><div><br></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><b style="font-size:small;color:rgb(51,51,51)">Trinh Nguyen</b><br></div><div><u style="font-size:12.8px;color:rgb(0,0,0)"><a href="https://www.edlab.xyz" target="_blank">www.edlab.xyz</a></u><br></div><div><br></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div>