[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...
More information about the Gnutls-devel