Skip to content

chore(docker): use APT nodejs for all Raspberry Pi boards#2678

Merged
nicomiguelino merged 1 commit intoScreenly:masterfrom
nicomiguelino:fix-docker-image-build/nodejs-version
Feb 24, 2026
Merged

chore(docker): use APT nodejs for all Raspberry Pi boards#2678
nicomiguelino merged 1 commit intoScreenly:masterfrom
nicomiguelino:fix-docker-image-build/nodejs-version

Conversation

@nicomiguelino
Copy link
Contributor

@nicomiguelino nicomiguelino commented Feb 24, 2026

Issues Fixed

Fixes a Docker image build failure on Raspberry Pi 3 and Pi 4 devices caused by the NodeSource setup script not supporting the armhf architecture.

Description

The NodeSource setup script only supports amd64 and arm64 architectures. Raspberry Pi 3 and Pi 4 use armhf, which caused the following error during Docker builds:

Error: Unsupported architecture: armhf. Only amd64, arm64 are supported.

Previously, pi1 and pi2 used APT-provided nodejs/npm, while pi3 and pi4 fell through to the NodeSource path. This PR extends the APT install branch to cover all Raspberry Pi boards (pi1, pi2, pi3, pi4). The NodeSource path is now only used for non-Pi boards (x86/x86_64).

Checklist

  • I have performed a self-review of my own code.
  • New and existing unit tests pass locally and on CI with my changes.
  • I have done an end-to-end test for Raspberry Pi devices.
  • I have tested my changes for x86 devices.
  • I added a documentation for the changes I have made (when necessary).

NodeSource doesn't support armhf architecture (used by pi3/pi4),
so fall back to APT-provided nodejs/npm for those boards.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@nicomiguelino nicomiguelino requested a review from a team as a code owner February 24, 2026 06:59
@sonarqubecloud
Copy link

@nicomiguelino nicomiguelino merged commit f0f6497 into Screenly:master Feb 24, 2026
7 checks passed
@nicomiguelino nicomiguelino deleted the fix-docker-image-build/nodejs-version branch February 24, 2026 07:13
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.

1 participant