[gnutls-devel] GnuTLS | .gitlab-ci.yml: replace valgrind checks with ASan (!1467)

[Read-only notification of GnuTLS library development activities]

Alexander Sosedkin commented:


re f23c3a6cba43706a6ebb3f9b0018cd658dcc0a72: good to have it!

re this MR: not particularly convinced on the time saving side.

Quoting select competing stage 3 job durations from the [latest pipeline of master at the time of writing](https://gitlab.com/gnutls/gnutls/-/pipelines/363372618/builds):
```
passed debian-cross/aarch64-linux-gnu/test   00:53:03
passed debian-cross/arm-linux-gnueabihf/test 00:50:56
failed fedora-notools/test                   03:00:02
passed fedora-static-analyzers/test          01:40:34
passed fedora-valgrind-aggressive/test       00:29:23
passed fedora-valgrind/test                  00:28:36
```

Doesn't look like valgrind jobs are on the critical path. If timeouts are caused by hanging tests and not raw performance considerations, I'd prefer test fixing and both checkks coexisting.

-- 
Reply to this email directly or view it on GitLab: https://gitlab.com/gnutls/gnutls/-/merge_requests/1467#note_669980027
You're receiving this email because of your account on gitlab.com.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.gnupg.org/pipermail/gnutls-devel/attachments/20210906/be8ec011/attachment.html>