Add check for nginx pid file in test docker images #1166

sean-breen · 2025-07-14T15:45:10Z

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

I have read the CONTRIBUTING document
I have run make install-tools and have attached any dependency changes to this pull request
If applicable, I have added tests that prove my fix is effective or that my feature works
If applicable, I have checked that any relevant tests pass after adding my changes
If applicable, I have updated any relevant documentation (README.md)
If applicable, I have tested my cross-platform changes on Ubuntu 22, Redhat 8, SUSE 15 and FreeBSD 13

add check for nginx pid file

a3d037c

sean-breen requested a review from a team as a code owner July 14, 2025 15:45

add alpine default path /var/run/nginx/nginx.pid

47034b0

oCHRISo added the v3.x Issues and Pull Requests related to the major version v3 label Jul 16, 2025

dhurley changed the title ~~add check for nginx pid file~~ Add check for nginx pid file in test dockerfiles Jul 16, 2025

dhurley changed the title ~~Add check for nginx pid file in test dockerfiles~~ Add check for nginx pid file in test docker images Jul 16, 2025

dhurley approved these changes Jul 16, 2025

View reviewed changes

aphralG approved these changes Jul 17, 2025

View reviewed changes

sean-breen merged commit 240cf41 into main Jul 17, 2025
25 checks passed

sean-breen deleted the entrypoint-fix-for-amd64 branch July 17, 2025 10:08

Provide feedback