Skip to content

Conversation

@sireeshajonnalagadda
Copy link
Contributor

@sireeshajonnalagadda sireeshajonnalagadda commented Oct 22, 2025

Feature:
Docker-in-docker

Description:

Solves issue
This is to add support of azureLinux OS for docker-in-docker feature

Changes:
Modified install script to work both ubuntu/debian and azureLinux
Created scenarios to check for both moby true and false

Checklist:
Changes worked as expected

install.sh compatibility
Confirmed that install.sh continues to work correctly with non–azure-linux base images, specifically Ubuntu and Debian.

Cross-distro testing
Ran scenarios with Ubuntu, Debian, and Mariner base images to validate docker-in-docker functionality across multiple environments.

Azure Linux support
Verified that after adding the implementation for the azure-linux distro, dev containers can be successfully built and run using docker-in-docker.

@sireeshajonnalagadda sireeshajonnalagadda marked this pull request as ready for review November 3, 2025 11:21
@sireeshajonnalagadda sireeshajonnalagadda requested a review from a team as a code owner November 3, 2025 11:21
@Mathiyarasy Mathiyarasy self-requested a review November 4, 2025 07:03
Copy link

@sam-byng sam-byng left a comment

Choose a reason for hiding this comment

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

Hey @sireeshajonnalagadda - thanks for the PR :)

In PR description:

Could you list the testing / verification that has been performed to confirm that docker-in-docker feature is still working as expected after the addition of azure-linux OS support?

In particular:

  • Is install.sh working with other base image types (not just azure-linux?)
  • is azure-linux working? / can we build a dev container using it with this feature?

This seems like a big change to a fundamental technology (install script for docker-in-docker) so want to make sure we're doing the right thing here.

@sireeshajonnalagadda
Copy link
Contributor Author

Hi @sam-byng,
I have modified the description to make it clearer and more meaningful.

@sireeshajonnalagadda sireeshajonnalagadda marked this pull request as ready for review November 26, 2025 11:57
@sam-byng
Copy link

Thanks - LGTM

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.

3 participants