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

Read-only notification of GnuTLS library development activities gnutls-devel at lists.gnutls.org
Mon Sep 6 10:59:38 CEST 2021

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>

More information about the Gnutls-devel mailing list