<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, 3 May 2017 at 01:24 Victor Stinner <<a href="mailto:victor.stinner@gmail.com">victor.stinner@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I spent last week working on fixing buildbots:<br>
<br>
   <a href="https://www.python.org/dev/buildbot/" rel="noreferrer" target="_blank">https://www.python.org/dev/buildbot/</a><br>
<br>
It should now be able again to rely on them to detect regressions. </blockquote><div><br></div><div>Thanks, Victor!</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Changes:<br>
<br>
* Fix various bugs (I don't even recall which ones)<br>
* Fix multiple random failures<br>
* Fix dozen of warnings<br>
<br>
I also enhanced the configuration of buildbots to search for warnings<br>
in the output of the "tests" step (Python test suite). I use a regular<br>
expression to search for patterns:<br>
<br>
<a href="https://github.com/python/buildmaster-config/blob/master/master/master.cfg#L93-L125" rel="noreferrer" target="_blank">https://github.com/python/buildmaster-config/blob/master/master/master.cfg#L93-L125</a><br>
<br>
If a warning is found, the buildbot now becomes orange. As I wrote, I<br>
fixed dozens of warnings, but there are still a few ones (and many<br>
unstable tests failing randomly).<br>
<br>
The orange state notify for example when a test failed once, but<br>
passed when it was run again in verbose mode. It helps to detect<br>
unstable tests, but also real bugs.<br>
<br>
<br>
The "Warning -- xxx was modified by test_xxx" warning now always log<br>
the old and new value to ease debugging these warnings. Example on<br>
Python 2.7:<br>
<br>
Warning -- files was modified by test_sax<br>
  Before: []<br>
  After:  ['@test_352_tmp']<br>
<br>
<br>
TODO:<br>
<br>
* Create a mailing list and send an email to this list when the state<br>
of a buildbot changes (only from green to orange or red? similar to<br>
what we have on #python-dev IRC channel). I suggest to start with a<br>
whitelist of buildbots known to be stable... hum, like the list of<br>
"stable buildbots" ;-) Let's start with the 3.x branch, and later<br>
enable it on more branches. The mailing list should help to coordinate<br>
when multiple developers work on buildbot issues in parallel.<br></blockquote><div><br></div><div>SGTM.</div><div><br></div><div>-Brett</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
* Fix FreeBSD buildbots which fail to compile Python because of<br>
<a href="http://bugs.python.org/issue23404" rel="noreferrer" target="_blank">http://bugs.python.org/issue23404</a> I proposed a change to not<br>
regenerate generated files based on file modification time anymore,<br>
but require an explicit action ("make regen-all"). This change solves<br>
many practical issues.<br>
<br>
* Fix remaining warnings.<br>
<br>
* Fix unstable tests.<br>
<br>
* Add more warnings! I proposed to emit ResourceWarning in<br>
multiprocessing.Queue and concurrent.futures executors ;-)<br>
<a href="http://bugs.python.org/issue30171" rel="noreferrer" target="_blank">http://bugs.python.org/issue30171</a> and<br>
<a href="http://bugs.python.org/issue30244" rel="noreferrer" target="_blank">http://bugs.python.org/issue30244</a><br>
<br>
* Add more buildbots! Zachary Ware proposed to add a buildbot running<br>
"regen-all" to check that generated files are up to date.<br>
<br>
* Repeat ;-)<br>
<br>
Victor<br>
_______________________________________________<br>
Python-Dev mailing list<br>
<a href="mailto:Python-Dev@python.org" target="_blank">Python-Dev@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/python-dev" rel="noreferrer" target="_blank">https://mail.python.org/mailman/listinfo/python-dev</a><br>
Unsubscribe: <a href="https://mail.python.org/mailman/options/python-dev/brett%40python.org" rel="noreferrer" target="_blank">https://mail.python.org/mailman/options/python-dev/brett%40python.org</a><br>
</blockquote></div></div>