Enable Valgrind in CI (#2297)

Using HTTP instead of HTTPS and
82107f4b6c improve test run-time so that
this is now feasible.
This commit is contained in:
Andrew Gaul 2023-08-29 07:11:26 -07:00 committed by GitHub
parent 7bb9609827
commit a2f2f72aaf
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 4 additions and 2 deletions

View File

@ -208,7 +208,6 @@ jobs:
# [NOTE]
# Currently following test is not supported:
# - sanitize_memory : Future support planned
# - valgrind : Requires more than an hour of testing time
#
matrix:
checktype:
@ -216,6 +215,7 @@ jobs:
- sanitize_address
- sanitize_others
- sanitize_thread
- valgrind
container:
image: fedora:38
@ -263,6 +263,7 @@ jobs:
echo "CXXFLAGS=${COMMON_CXXFLAGS} -O1" >> $GITHUB_ENV
echo 'VALGRIND=--leak-check=full' >> $GITHUB_ENV
echo 'RETRIES=100' >> $GITHUB_ENV
echo 'S3_URL=http://127.0.0.1:8081' >> $GITHUB_ENV
fi
- name: Build

View File

@ -61,7 +61,7 @@ ALL_TESTS=1 make check -C test/
make clean
./configure CXXFLAGS="$COMMON_FLAGS"
make --jobs="$(nproc)"
ALL_TESTS=1 RETRIES=100 VALGRIND="--leak-check=full --error-exitcode=1" make check -C test/
ALL_TESTS=1 RETRIES=100 VALGRIND="--leak-check=full --error-exitcode=1" S3_URL=http://127.0.0.1:8081 make check -C test/
#
# Local variables:

View File

@ -1,3 +1,4 @@
s3proxy.endpoint=http://127.0.0.1:8081
s3proxy.secure-endpoint=https://127.0.0.1:8080
s3proxy.authorization=aws-v2-or-v4
s3proxy.identity=local-identity