-
Notifications
You must be signed in to change notification settings - Fork 400
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support docker-in-docker on Ubuntu Noble (24.04) #969
Conversation
@microsoft-github-policy-service agree company=Elefant-Ai |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This makes sense, left one minor comment. Thanks!
DOCKER_MOBY_ARCHIVE_VERSION_CODENAMES="bookworm buster bullseye bionic focal jammy" | ||
DOCKER_LICENSED_ARCHIVE_VERSION_CODENAMES="bookworm buster bullseye bionic focal hirsute impish jammy" | ||
DOCKER_MOBY_ARCHIVE_VERSION_CODENAMES="bookworm buster bullseye bionic focal jammy noble" | ||
DOCKER_LICENSED_ARCHIVE_VERSION_CODENAMES="bookworm buster bullseye bionic focal hirsute impish jammy noble" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you help add a similar test 👇 for the noble
image? Thanks!
features/test/docker-in-docker/scenarios.json
Lines 2 to 10 in 18c1823
"docker_build_fallback_compose": { | |
"image": "ubuntu:focal", | |
"features": { | |
"docker-in-docker": { | |
"version": "latest", | |
"dockerDashComposeVersion": "latest" | |
} | |
} | |
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Better idea, I think we should update these 👇 workflows and add the mcr.microsoft.com/devcontainers/base:noble
image to the baseImage
features/.github/workflows/test-all.yaml
Lines 43 to 51 in 18c1823
baseImage: | |
[ | |
"ubuntu:focal", | |
"ubuntu:jammy", | |
"debian:11", | |
"debian:12", | |
"mcr.microsoft.com/devcontainers/base:ubuntu", | |
"mcr.microsoft.com/devcontainers/base:debian", | |
] |
features/.github/workflows/test-pr.yaml
Lines 50 to 58 in 18c1823
baseImage: | |
[ | |
"ubuntu:focal", | |
"ubuntu:jammy", | |
"debian:11", | |
"debian:12", | |
"mcr.microsoft.com/devcontainers/base:ubuntu", | |
"mcr.microsoft.com/devcontainers/base:debian", | |
] |
This tiny changed seem to work for me.
https://github.com/jjh42/docker-in-docker-noble
as a temporary workaround you can use
"ghcr.io/jjh42/docker-in-docker-noble/docker-in-docker:latest": {},
instead of the official docker-in-docker feature.