Skip to content

fix(api): run the API on Node 24, matching its build - #666

Merged
DerekRoberts merged 3 commits into
mainfrom
fix/api-runtime-node24
Oct 11, 2026
Merged

DerekRoberts merged 3 commits into
mainfrom
fix/api-runtime-node24

Conversation

@DerekRoberts

@DerekRoberts DerekRoberts commented Oct 9, 2026 •

Copy link
Copy Markdown
Member

The API image installs its dependencies with Node 24 but runs them on the Node 22 distroless image, so the runtime is a different major version from the one CI tests on.

Refs #597

Changes

  • The API runtime stage uses gcr.io/distroless/nodejs24-debian12:nonroot, the image quickstart-openshift's backend uses. It now matches the node:24 build stage and the Node 24 that the API Tests job runs on.
  • The image builds locally, starts on Node 24.14.0 and tries to connect to MongoDB as before. The PR deployment and Cypress tests exercise it in CI.

Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

The runtime stage used the Node 22 distroless image while dependencies were installed and tested on Node 24.

Refs #597
Copilot AI balanced review requested due to automatic review settings October 9, 2026 23:09
@DerekRoberts DerekRoberts self-assigned this Oct 9, 2026

Copilot AI 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.

🟢 Approval recommended

The focused image update correctly resolves the build/runtime Node.js version mismatch.

0 open findings

What changed in this PR

Aligns the API runtime with its Node 24 build and CI environment.

Changes:

  • Updates the distroless runtime image from Node 22 to Node 24 on Debian 12.
File Description
api/​Dockerfile Uses the Node 24 distroless runtime image.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@DerekRoberts DerekRoberts mentioned this pull request Oct 9, 2026
3 of 8 tasks
@DerekRoberts
DerekRoberts merged commit 65d006c into main Oct 11, 2026
19 checks passed
@DerekRoberts
DerekRoberts deleted the fix/api-runtime-node24 branch October 11, 2026 02:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants