Skip to content
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

feat: add devcontainer #2131

Merged
merged 3 commits into from
Jan 12, 2025
Merged

feat: add devcontainer #2131

merged 3 commits into from
Jan 12, 2025

Conversation

GarciaLnk
Copy link
Contributor

Fixes #2046

I've confirmed that the following recipes are working properly inside the devcontainer:

just build
just run
just build-iso
just run-iso

in docker, images are tagged to docker.io by default, this makes the behaviour consistent with podman, where localhost is used instead
fallback to 'missing' since podman's 'newer' is not available in docker
tested working recipes in devcontainer:
* just build
* just run
* just build-iso
* just run-iso
@GarciaLnk GarciaLnk requested a review from castrojo as a code owner January 9, 2025 22:43
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. enhancement New feature or request labels Jan 9, 2025
@m2Giles
Copy link
Member

m2Giles commented Jan 9, 2025

Also need to check rechunk. But it should work with docker out of docker.

Did you try Docker in Docker? I think this one is probably a footgun with people trying to do this inside of a rootless podman container though.

@GarciaLnk
Copy link
Contributor Author

GarciaLnk commented Jan 10, 2025

Also need to check rechunk. But it should work with docker out of docker.

just build-rechunk fails with docker because docker mount doesn't exist.

Did you try Docker in Docker? I think this one is probably a footgun with people trying to do this inside of a rootless podman container though.

Docker in docker throws a bunch of permissions errors, I didn't dig too deep though.

@castrojo castrojo enabled auto-merge January 12, 2025 13:28
Copy link
Member

@castrojo castrojo left a comment

Choose a reason for hiding this comment

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

Thank You!

@castrojo castrojo added this pull request to the merge queue Jan 12, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 12, 2025
Merged via the queue into ublue-os:main with commit ce84b9d Jan 12, 2025
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Readd devcontainer support
3 participants