-
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
Allow moby-buildx version to be specified in devcontainer.json #838
Allow moby-buildx version to be specified in devcontainer.json #838
Conversation
@microsoft-github-policy-service agree company="Rover.com" |
I think I also need to bump the version here, but I'm unsure if you'd consider this a minor or patch bump |
The idea here being that folks can use |
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.
Thank you for the quick fix, left some feedback
- Specify mobyBuildxVersion in devcontainer-feature.json with pinned and commented default value - Bump minor version - Leverage default value in install.sh - Only set buildx_version_suffix when installing moby
Ah, looks like I need to handle different distributions. Give me a few more mins |
@samruddhikhandale Let's see what GHA has to say about the latest commit. It's a little ugly since it's effectively the same implementation as the suffix-finding code for the docker engine version, but hopefully these can all get a refactor into a common function later ? |
@samruddhikhandale Whoops, had an extra line that'll make the current workflow runs likely fail. Just pushed again |
@sayhiben would be interested in making similar changes once this PR gets approved to
Yep, this is alright for now, I can refactor it when I remove the I'll merge this PR when all tests are green. Thank you so much, we appreciate it 👏 |
Hey, not sure that is same issue or not, but I'm trying to apply the fix (for
|
Fixes #837