Skip to content

Conversation

@S1ealth
Copy link
Contributor

@S1ealth S1ealth commented Jun 26, 2025

Proposed changes

Will fix #321

Describe the use case and detail of the change. If this PR addresses an issue on GitHub, make sure to include a link to that issue using one of the supported keywords here in this description (not in the title of the PR).

Checklist

Before creating a PR, run through this checklist and mark each as complete:

  • I have read the contributing guidelines
  • I have signed the F5 Contributor License Agreement (CLA)
  • I have run the update.sh script and ensured all entrypoint/Dockerfile template changes have been applied to the relevant image entrypoint scripts & Dockerfiles
  • I have tested that the NGINX Docker unprivileged image builds and runs correctly on all supported architectures on an unprivileged environment (check out the README for more details)
  • I have updated any relevant documentation (README.md)

@S1ealth S1ealth requested a review from a team as a code owner June 26, 2025 12:15
@github-actions
Copy link

github-actions bot commented Jun 26, 2025

✅ All required contributors have signed the F5 CLA for this PR. Thank you!
Posted by the CLA Assistant Lite bot.

@S1ealth
Copy link
Contributor Author

S1ealth commented Jun 26, 2025

I have hereby read the F5 CLA and agree to its terms

@S1ealth S1ealth changed the base branch from update to main June 26, 2025 13:06
@S1ealth S1ealth force-pushed the fix--pid-file-location branch from d819664 to 577349e Compare June 26, 2025 13:16
@alessfg alessfg changed the title fix: pid file location in init.d script fix: PID file location in init.d script Jun 26, 2025
@alessfg alessfg added the bug Something isn't working label Jun 26, 2025
@alessfg alessfg merged commit 4f7239f into nginx:main Jun 27, 2025
19 of 20 checks passed
@S1ealth S1ealth deleted the fix--pid-file-location branch June 27, 2025 12:43
S1ealth added a commit to S1ealth/docker-nginx-unprivileged that referenced this pull request Jun 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

healthcheck with service command always returns unhealthy status

2 participants