Skip to content

build: upgrade nginx base image to 1.29.8#53

Merged
shimoncohen merged 1 commit into
masterfrom
feat/upgrade-nginx-1.29
Jul 20, 2026
Merged

build: upgrade nginx base image to 1.29.8#53
shimoncohen merged 1 commit into
masterfrom
feat/upgrade-nginx-1.29

Conversation

@shimoncohen

Copy link
Copy Markdown
Contributor

What

Bump the base image from nginxinc/nginx-unprivileged:1.28.0-alpine3.21-otel to 1.29.8-alpine3.23-otel.

Why

  • Newer nginx runtime + OpenTelemetry module.
  • nginx ≥ 1.27.3 is required for server … resolve; runtime DNS re-resolution used by downstream consumers (nginx-s3-gateway).
  • Unblocks the nginx-s3-gateway "reuse this base image" work.

Verification

  • docker build green; nginx -vnginx/1.29.8.
  • auth.js + status_site.conf unchanged and present.
  • njs import of auth.js resolves; opaAuth / jwtPayloadSub export as functions.
  • With the otel module loaded (as at runtime), otel_trace is recognized and a full config nginx -t passes.

Note

Only the FROM line changes. On merge + release, CI publishes common/nginx:v<release> carrying nginx 1.29.8 — downstream should pin to that release tag.

Bump the base from nginx-unprivileged 1.28.0-alpine3.21-otel to
1.29.8-alpine3.23-otel. Provides the newer nginx runtime + otel module
(nginx >= 1.27.3 required for `server ... resolve;` DNS re-resolution used
downstream). auth.js and status_site.conf are unchanged and validated
against 1.29.8 (njs import ok; otel_trace recognized).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

@razbroc razbroc left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please test new versuion

@shimoncohen
shimoncohen merged commit 12725ef into master Jul 20, 2026
2 checks passed
@shimoncohen
shimoncohen deleted the feat/upgrade-nginx-1.29 branch July 20, 2026 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants