<div dir="auto"><div>If ASAN is too slow to run in the CI should we restore the previous homemade checks for memory leaks as an alternative to be run in regular CI runs and leave ASAN builds to run-checker on the master branch only? </div><div dir="auto"><br></div><div dir="auto">Here is another idea that would be interesting if we restore the previous checks:</div><div dir="auto"></div><div dir="auto">I don't know what kind of options github offers on this, but would it be possible to run triggered CI on something that is not Travis and does not timeout and still have the results in the PR?</div><div dir="auto">If something like that would be possible we could move the ASAN builds to extended_tests, rely on the previous memleak detection for the regular CI runs, and then trigger with a script or Github Action the extended_tests when the approval:done label is added. </div><div dir="auto"><br></div><div dir="auto">That way, by the time something is ready to be merged we should have a full picture! </div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">Nicola<br><br><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">On Wed, Feb 5, 2020, 10:25 Matt Caswell <<a href="mailto:matt@openssl.org">matt@openssl.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Since we fixed the Travis builds 4 out of the 8 builds on master that<br>
have taken place have errored due to a timeout.<br>
<br>
The msan build is consistently taking a *very* long time to run. If it<br>
gets to 50 minutes then Travis cuts it off and the build fails.<br>
<br>
Should we disable the msan build?<br>
<br>
Matt<br>
<br>
<br>
-------- Forwarded Message --------<br>
Subject: Errored: openssl/openssl#31939 (master - 34b1676)<br>
Date: Wed, 05 Feb 2020 00:02:01 +0000<br>
From: Travis CI <<a href="mailto:builds@travis-ci.org" target="_blank" rel="noreferrer">builds@travis-ci.org</a>><br>
To: <a href="mailto:openssl-commits@openssl.org" target="_blank" rel="noreferrer">openssl-commits@openssl.org</a><br>
<br>
<br>
<br>
openssl<br>
<br>
/<br>
<br>
openssl<br>
<br>
<<a href="https://travis-ci.org/openssl/openssl?utm_medium=notification&utm_source=email" rel="noreferrer noreferrer" target="_blank">https://travis-ci.org/openssl/openssl?utm_medium=notification&utm_source=email</a>><br>
<br>
<br>
branch iconmaster <<a href="https://github.com/openssl/openssl/tree/master" rel="noreferrer noreferrer" target="_blank">https://github.com/openssl/openssl/tree/master</a>><br>
<br>
build has errored<br>
Build #31939 has errored<br>
<<a href="https://travis-ci.org/openssl/openssl/builds/646181069?utm_medium=notification&utm_source=email" rel="noreferrer noreferrer" target="_blank">https://travis-ci.org/openssl/openssl/builds/646181069?utm_medium=notification&utm_source=email</a>><br>
arrow to build time<br>
clock icon50 mins and 3 secs<br>
<br>
Pauli avatarPauli<br>
<br>
34b1676 CHANGESET →<br>
<<a href="https://github.com/openssl/openssl/compare/e3b1ccad694a...34b167625af5" rel="noreferrer noreferrer" target="_blank">https://github.com/openssl/openssl/compare/e3b1ccad694a...34b167625af5</a>><br>
<br>
Make minimum size for secure memory a size_t.<br>
<br>
The minimum size argument to CRYPTO_secure_malloc_init() was an int but<br>
ought<br>
to be a size_t since it is a size.<br>
<br>
>From an API perspective, this is a change. However, the minimum size is<br>
verified as being a positive power of two and it will typically be a small<br>
constant.<br>
<br>
Reviewed-by: David von Oheimb <<a href="mailto:david.von.oheimb@siemens.com" target="_blank" rel="noreferrer">david.von.oheimb@siemens.com</a>><br>
(Merged from #11003)<br>
<br>
Want to know about upcoming build environment updates?<br>
<br>
Would you like to stay up-to-date with the upcoming Travis CI build<br>
environment updates? We set up a mailing list for you!<br>
<br>
SIGN UP HERE <<a href="http://eepurl.com/9OCsP" rel="noreferrer noreferrer" target="_blank">http://eepurl.com/9OCsP</a>><br>
<br>
book icon<br>
<br>
Documentation <<a href="https://docs.travis-ci.com/" rel="noreferrer noreferrer" target="_blank">https://docs.travis-ci.com/</a>> about Travis CI<br>
<br>
Have any questions? We're here to help. <mailto:<a href="mailto:support@travis-ci.com" target="_blank" rel="noreferrer">support@travis-ci.com</a>><br>
Unsubscribe<br>
<<a href="https://travis-ci.org/account/preferences/unsubscribe?repository=5849220&utm_medium=notification&utm_source=email" rel="noreferrer noreferrer" target="_blank">https://travis-ci.org/account/preferences/unsubscribe?repository=5849220&utm_medium=notification&utm_source=email</a>><br>
from build emails from the openssl/openssl repository.<br>
To unsubscribe from *all* build emails, please update your settings<br>
<<a href="https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notification&utm_source=email" rel="noreferrer noreferrer" target="_blank">https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notification&utm_source=email</a>>.<br>
<br>
black and white travis ci logo <<a href="https://travis-ci.com" rel="noreferrer noreferrer" target="_blank">https://travis-ci.com</a>><br>
<br>
Travis CI GmbH, Rigaer Str. 8, 10427 Berlin, Germany | GF/CEO: Randy<br>
Jacops | Contact: <a href="mailto:contact@travis-ci.com" target="_blank" rel="noreferrer">contact@travis-ci.com</a> <mailto:<a href="mailto:contact@travis-ci.com" target="_blank" rel="noreferrer">contact@travis-ci.com</a>> |<br>
Amtsgericht Charlottenburg, Berlin, HRB 140133 B | Umsatzsteuer-ID gemäß<br>
§27 a Umsatzsteuergesetz: DE282002648<br>
<br>
</blockquote></div></div></div>