Commit 418efad
authored
Fix: add node-gpy to fix arm64 builds (#1616)
## Summary
- Add `node-gyp` to `apk add` in `ui/Dockerfile` and `python/Dockerfile`
— npm 11 on arm64 Wolfi no longer bundles it, causing `npm ci`/`npm
install` to fail with `Cannot find module 'node-gyp/bin/node-gyp.js'`
## Root cause
`ui/Dockerfile` uses `FROM --platform=$BUILDPLATFORM`, so the deps stage
runs natively on the build host. On Apple Silicon (arm64), Wolfi's
`nodejs~24` ships with npm 11, which no longer bundles `node-gyp` but
still requires it internally. This only affects arm64 builds — GitHub CI
runs on amd64 where the Wolfi nodejs package ships with npm 10.
## Testing
Deployed via `make helm-install` on Apple Silicon and verified that e2e
tests pass.
Signed-off-by: Jeremy Alvis <jeremy.alvis@solo.io>1 parent fcad888 commit 418efad
2 files changed
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
62 | 62 | | |
63 | 63 | | |
64 | 64 | | |
65 | | - | |
| 65 | + | |
66 | 66 | | |
67 | 67 | | |
68 | 68 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
13 | | - | |
| 13 | + | |
14 | 14 | | |
15 | 15 | | |
16 | 16 | | |
| |||
0 commit comments