Skip to content

Conversation

@S1ealth
Copy link
Contributor

@S1ealth S1ealth commented Jun 30, 2025

Proposed changes

I'm very sorry for the extra PR - in #323 i updated only template file - i thought github action will run ./update.sh, so i didn't commit new dockerfile's 😞
This PR will update them.

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 30, 2025 12:37
@alessfg
Copy link
Member

alessfg commented Jun 30, 2025

Somehow this slipped through my review too! Thanks for the quick fix!

@alessfg alessfg added the bug Something isn't working label Jun 30, 2025
@alessfg alessfg enabled auto-merge June 30, 2025 17:01
@alessfg alessfg added this pull request to the merge queue Jun 30, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Jun 30, 2025
@alessfg alessfg added this pull request to the merge queue Jun 30, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Jul 1, 2025
@alessfg alessfg merged commit e9196bb into nginx:main Jul 1, 2025
19 of 20 checks passed
@alessfg alessfg changed the title chore: update dockerfiles to match template from #323 chore: Update Dockerfiles to match template from #323 Jul 1, 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.

2 participants