CI: Replace the requirement for 'sudo' with a call to 'ulimit -n'

Using 'sudo' required quite a few workarounds in various places. Setting an
explicit 'ulimit -n' removes the requirement for 'sudo', resulting in a cleaner
workflow configuration.
This commit is contained in:
Tim Duesterhus 2021-06-13 15:02:24 +02:00 committed by Willy Tarreau
parent 213bb99f9e
commit 8a91fa4219

View File

@ -99,13 +99,14 @@ jobs:
run: echo "::add-matcher::.github/vtest.json" run: echo "::add-matcher::.github/vtest.json"
- name: Run VTest for HAProxy ${{ steps.show-version.outputs.version }} - name: Run VTest for HAProxy ${{ steps.show-version.outputs.version }}
id: vtest id: vtest
# sudo is required, because macOS fails due to an open files limit. run: |
run: sudo make reg-tests VTEST_PROGRAM=../vtest/vtest REGTESTS_TYPES=default,bug,devel # This is required for macOS which does not actually allow to increase
# the '-n' soft limit to the hard limit, thus failing to run.
ulimit -n 5000
make reg-tests VTEST_PROGRAM=../vtest/vtest REGTESTS_TYPES=default,bug,devel
- name: Show results - name: Show results
if: ${{ failure() }} if: ${{ failure() }}
# The chmod / sudo is necessary due to the `sudo` while running the tests.
run: | run: |
sudo chmod a+rX ${TMPDIR}/haregtests-*/
for folder in ${TMPDIR}/haregtests-*/vtc.*; do for folder in ${TMPDIR}/haregtests-*/vtc.*; do
printf "::group::" printf "::group::"
cat $folder/INFO cat $folder/INFO
@ -115,6 +116,6 @@ jobs:
shopt -s nullglob shopt -s nullglob
for asan in asan.log*; do for asan in asan.log*; do
echo "::group::$asan" echo "::group::$asan"
sudo cat $asan cat $asan
echo "::endgroup::" echo "::endgroup::"
done done