Skip to content

Set TEST_NON_ROOT_USER in CI #19290

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: PHP-8.4
Choose a base branch
from

Conversation

arnaud-lb
Copy link
Member

@arnaud-lb arnaud-lb commented Jul 29, 2025

Some times ago I added a few opcache.preload_user tests that need a TEST_NON_ROOT_USER variable, but I never defined this variable in CI (these tests also need to run as root, but there were no CI job that ran as root at the time).

Here I fix this by setting the variable in the ALPINE nightly job, which runs as root.

Also set ASAN_OPTIONS=detect_leaks=0 for these tests, because the preload child process doesn't exit cleanly:

exit(ret == SUCCESS ? 0 : 1);

Targeting 8.4 because .github/actions/test-alpine does not exist in earlier branches.

@arnaud-lb arnaud-lb marked this pull request as ready for review July 29, 2025 16:03
@arnaud-lb arnaud-lb force-pushed the ci-set-TEST_NON_ROOT_USER branch from 440700d to d6659a2 Compare July 29, 2025 16:19
@arnaud-lb arnaud-lb changed the base branch from master to PHP-8.4 July 29, 2025 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant