<div dir="ltr">On Mon, Oct 3, 2016 at 12:11 PM, Matthias Kuhn <span dir="ltr"><<a href="mailto:matthias@opengis.ch" target="_blank">matthias@opengis.ch</a>></span> wrote:<br><div class="gmail_extra"><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"><span><br>
<br>
On 10/03/2016 12:04 PM, Alessandro Pasotti wrote:<br>
> On Mon, Oct 3, 2016 at 11:28 AM, Nyall Dawson <<a href="mailto:nyall.dawson@gmail.com" target="_blank">nyall.dawson@gmail.com</a><br>
</span><span>> <mailto:<a href="mailto:nyall.dawson@gmail.com" target="_blank">nyall.dawson@gmail.com</a><wbr>>> wrote:<br>
><br>
> On 3 October 2016 at 19:11, Alessandro Pasotti <<a href="mailto:apasotti@gmail.com" target="_blank">apasotti@gmail.com</a><br>
</span>> <mailto:<a href="mailto:apasotti@gmail.com" target="_blank">apasotti@gmail.com</a>>> wrote:<br>
<div><div class="gmail-m_4606763823010120323h5">> > On Mon, Oct 3, 2016 at 10:44 AM, Matthias Kuhn <<a href="mailto:matthias@opengis.ch" target="_blank">matthias@opengis.ch</a> <mailto:<a href="mailto:matthias@opengis.ch" target="_blank">matthias@opengis.ch</a>>> wrote:<br>
> >><br>
> >> Following the conversation from<br>
> >> <a href="https://github.com/qgis/QGIS/commit/89d4998" rel="noreferrer" target="_blank">https://github.com/qgis/QGIS/c<wbr>ommit/89d4998</a><br>
> <<a href="https://github.com/qgis/QGIS/commit/89d4998" rel="noreferrer" target="_blank">https://github.com/qgis/<wbr>QGIS/commit/89d4998</a>> , this is an announcement<br>
> >> to notify that the PyQgsServerWFST has been disabled on the Travis builds.<br>
> >><br>
> >> Work has been underway to remove "noisy" tests from Travis which cause<br>
> >> false alarms and these tests regularly fail for unrelated changes.<br>
> >> False alarms are bad as they undermine confidence in the CI results<br>
> >> and encourage contributors to ignore the test results.<br>
> >><br>
> >> If you're interested in seeing these tests continue as part of the CI<br>
> >> infrastructure here are some examples of typical random failures these<br>
> >> tests encounter:<br>
> >><br>
> >> PyQgsServerWFST: <a href="https://travis-ci.org/qgis/QGIS/builds/164559513" rel="noreferrer" target="_blank">https://travis-ci.org/qgis/QGI<wbr>S/builds/164559513</a><br>
> <<a href="https://travis-ci.org/qgis/QGIS/builds/164559513" rel="noreferrer" target="_blank">https://travis-ci.org/qgis/Q<wbr>GIS/builds/164559513</a>><br>
> >><br>
> >> Matthias<br>
> ><br>
> ><br>
> ><br>
> > Hi Matthias,<br>
> ><br>
> > Is there any way to see the complete build logs for that particular build?<br>
> ><br>
> > I'm trying with dash but the interesting part was apparently cut.<br>
> ><br>
> > The rest of the test output was removed since it exceeds the threshold of<br>
> > 307200 bytes.<br>
><br>
> If you make the very first output from the test a 'CTEST_FULL_OUTPUT'<br>
> line, then the truncation doesn't occur.<br>
><br>
> test_qgsserver_accesscontrol.<wbr>py uses this, have a look how it's done<br>
> there.<br>
><br>
> Nyall<br>
><br>
><br>
><br>
> Thanks! I will do it.<br>
><br>
> BTW, restarting the build<br>
> <a href="https://travis-ci.org/qgis/QGIS/builds/164559513" rel="noreferrer" target="_blank">https://travis-ci.org/qgis/QGI<wbr>S/builds/164559513</a>, the WFS-T test now<br>
> passes but there is another one that fails:<br>
> <a href="https://travis-ci.org/qgis/QGIS/builds/164559513" rel="noreferrer" target="_blank">https://travis-ci.org/qgis/QGI<wbr>S/builds/164559513</a><br>
><br>
> PyQgsComposerPicture ....................***Timeout 60.00 sec<br>
><br>
> Should we disable that one too?<br>
<br>
</div></div>It's already disabled, the regex for blacklisting was broken and it was<br>
executed by mistake.<br>
Fixed in 8e54acf<br>
<br>
Matthias<br>
<span class="gmail-m_4606763823010120323HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>Hi,<br></div><div><br></div><div>I cannot see any other reason for the random failures than the "sleep(x)" that the test waited for the server to accept connections.<br><br></div><div>I've changed the check to a more robust wait-until-ready-or-timeout function, this should eliminate those odd sparse failures. <br></div></div><br><br></div><div class="gmail_extra">Please let me know if you still have problems with these tests.<br><br><a href="https://github.com/qgis/QGIS/pull/3563">https://github.com/qgis/QGIS/pull/3563</a><br><br></div><div class="gmail_extra"><br>-- <br><div class="gmail-m_4606763823010120323gmail_signature">Alessandro Pasotti<br>w3: <a href="http://www.itopen.it" target="_blank">www.itopen.it</a></div>
</div></div>