Skip to content

fix(build): avoid managed Node runtime in Alpine images - #7461

Merged
c121914yu merged 1 commit into
labring:mainfrom
DigHuang:codex/fix-volume-manager-ci
Aug 6, 2026
Merged

fix(build): avoid managed Node runtime in Alpine images#7461
c121914yu merged 1 commit into
labring:mainfrom
DigHuang:codex/fix-volume-manager-ci

Conversation

@DigHuang

@DigHuang DigHuang commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • replace the root devEngines runtime download with a Node.js >=22.22.0 engine requirement
  • refresh pnpm-lock.yaml to remove the managed node@runtime package
  • keep existing Alpine Dockerfiles unchanged

Problem

pnpm binds workspace scripts to the downloaded devEngines runtime. The official Node.js runtime artifact is not usable in the Alpine/musl builder, leaving the node shim pointed at a missing executable and breaking the volume-manager image build.

Verification

  • built projects/volume-manager/Dockerfile successfully with frozen lockfile on arm64
  • built the same image successfully for linux/amd64 with Docker Buildx
  • reran the Docker build after rebasing onto the latest upstream/main
  • git diff --check

@c121914yu
c121914yu merged commit 0591a4f into labring:main Aug 6, 2026
9 checks passed
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 24.6% 2206 / 8967
🔵 Statements 24.59% 2320 / 9431
🔵 Functions 20.97% 457 / 2179
🔵 Branches 19.93% 1019 / 5112
File CoverageNo changed files found.
Generated in workflow #2033 for commit 4d66934 by the Vitest Coverage Report Action

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

Admin Preview Image Ready!

ghcr.io/labring/fastgpt-pr:admin_4d66934b1afa3a4b1aa78e86bbd0fb3585df45a0

🕒 Time: 2026-08-06 11:24:40 (UTC+8)

@DigHuang
DigHuang deleted the codex/fix-volume-manager-ci branch August 6, 2026 03:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants