ci: Drop LSAN suppressions for PAM tests

These are not needed anymore as the libPAM leaks are fixed in noble
ubuntu · Nov 18, 2024 · 671525e · 671525e
1 parent d4948a4
commit 671525e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 18 deletions.
diff --git a/.github/workflows/qa.yaml b/.github/workflows/qa.yaml
@@ -257,13 +257,11 @@ jobs:
           echo "Running PAM integration tests"
           pushd ./pam/integration-tests
           go test -asan -gcflags=all="${GO_GC_FLAGS}" -c
-          # FIXME: Suppression may be removed with newer libpam, as the one we ship in ubuntu as some leaks
-          LSAN_OPTIONS=suppressions=$(pwd)/lsan.supp \
-            go tool test2json -p pam/integrations-test ./integration-tests.test \
-              -test.v=test2json \
-              -test.timeout ${GO_TESTS_TIMEOUT} | \
-            ../../tools/gotestfmt --logfile "${AUTHD_TEST_ARTIFACTS_PATH}/gotestfmt.pam-integration-tests-asan.log" || \
-            exit_code=$?
+          go tool test2json -p pam/integrations-test ./integration-tests.test \
+            -test.v=test2json \
+            -test.timeout ${GO_TESTS_TIMEOUT} | \
+          ../../tools/gotestfmt --logfile "${AUTHD_TEST_ARTIFACTS_PATH}/gotestfmt.pam-integration-tests-asan.log" || \
+          exit_code=$?
           popd
 
           # We don't need the xtrace output after this point

diff --git a/pam/integration-tests/lsan.supp b/pam/integration-tests/lsan.supp