<html><head></head><body>Hi,<br><br>With all due respect, this fixation on Gitlab CI seems delusional to me. It took several weeks for anybody to report an obvious and universal memory leak in the unit tests. Because nobody uses the leak sanitizer on regular basis except me.<br><br>As for the video problem, it only occurs in smoke testing not in unit testing, so any CI is entirely helpless there.<br><br>It's rather ironic that I was the one to object against adding the sanitizers to the configure script, and I'm semingly the only one to use them systematically in the end.<br><br>Besides premerge CI will fail in so many other ways that it's not even funny. It's great for an admin who has the power to fix the CI system or to overrule it. It sucks big time for everyone else who gets stuck whenever:<br>- one platform is down,<br>- contrib or sync fails due to network issue,<br>- build fails due to transient out-of-ressource situation,<br>- nondeterministic test case breaks pseudorandomly,<br>- system update introduces unrelated regressions (especially likely with leaksan).<br>And if it's gating, then it'll be miserable for people working in different continent or hobbyists, who'll have nobody to get live support from.<br><br><br>No, the only way to fix this is that developers use sanitizers, which was the whole point of adding them to configure.<br><br><div class="gmail_quote">Le 20 octobre 2019 21:01:25 GMT+03:00, Jean-Baptiste Kempf <jb@videolan.org> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<pre class="k9mail">Yo,<br><br>On Sun, Oct 20, 2019, at 15:33, Rémi Denis-Courmont wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">The points are that some people ostensibly don't run the test suite, and/or <br>smoke tests and/or enable the ADDR and UB sanitisers, causing easily avoidable <br>regressions in master.<br></blockquote><br>Agreed.<br><br>Breaking the build or breaking tests should not happen anymore. This is not normal.<br><br>We will then move to gitlab to get the CI to run pre-merge, as soon as possible.<br>CI should catch all of those.<br></pre></blockquote></div><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>