Skip to content

fix: add universal image to build and push list#329

Merged
matifali merged 2 commits intomainfrom
fix/add-universal-to-images
Apr 10, 2026
Merged

fix: add universal image to build and push list#329
matifali merged 2 commits intomainfrom
fix/add-universal-to-images

Conversation

@matifali
Copy link
Copy Markdown
Member

@matifali matifali commented Apr 10, 2026

The universal image was added in #307 but never included in the IMAGES array in scripts/images.sh. Both build_images.sh and push_images.sh iterate over this array, so the universal image was never built or pushed to Docker Hub.

Changes

  • Add "universal" to the IMAGES array in scripts/images.sh.
  • Add per-image platform override support via .platforms file — if present in the image directory, its content is used as the --platform value instead of the default.
  • Add images/universal/.platforms with linux/amd64 only, since mcr.microsoft.com/devcontainers/universal:linux has no arm64 variant.

🤖 This PR was created with the help of Coder Agents, and needs a human review. 🧑💻

The universal image was added in #307 but never included in
the IMAGES array in scripts/images.sh, so it was never built
or pushed to Docker Hub.
The devcontainers/universal base image is amd64-only. Add support
for a .platforms file per image directory to override the default
multi-arch platform list.
@matifali matifali force-pushed the fix/add-universal-to-images branch from 936d01d to c86d5c9 Compare April 10, 2026 12:03
@matifali
Copy link
Copy Markdown
Member Author

the upstream has no arm64 support: devcontainers/images#1604

@matifali matifali requested a review from johnstcn April 10, 2026 12:13
Copy link
Copy Markdown
Member

@johnstcn johnstcn left a comment

Choose a reason for hiding this comment

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

In future it might be worth investigating if we can make an equivalent image ourselves that has amd64 support.

Copy link
Copy Markdown
Member Author

Yeah Thats possible our dogfood image is the closest thing we have already that be repurposed.

@matifali matifali merged commit 2ab49fd into main Apr 10, 2026
2 checks passed
@matifali matifali deleted the fix/add-universal-to-images branch April 10, 2026 12:47
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.

2 participants