Skip to content

Add build-devcontainers.yaml and build-devcontainer.yaml workflows#530

Open
trxcllnt wants to merge 6 commits into
mainfrom
fea/build-devcontainer
Open

Add build-devcontainers.yaml and build-devcontainer.yaml workflows#530
trxcllnt wants to merge 6 commits into
mainfrom
fea/build-devcontainer

Conversation

@trxcllnt
Copy link
Copy Markdown
Contributor

@trxcllnt trxcllnt commented Apr 30, 2026

Add workflows to build and publish devcontainers to ghcr.io so we can use cacheFrom to speed up devcontainer launch times.

Depends on rapidsai/shared-actions#104

@trxcllnt trxcllnt added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Apr 30, 2026
@trxcllnt trxcllnt requested review from bdice and removed request for msarahan May 4, 2026 17:52
For example, if a repository has directories '.devcontainer/cuda12.9-pip/' and '.devcontainer/cuda13.1-pip/',
'["12.9", "13.1"]' could be passed here to build both of those devcontainers in CI.
type: string
default: '["12.9", "13.1"]'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a note that this may need to be changed. We're in the process of updating 13.1 to 13.2. Not all repos have the same devcontainers right now, depending on whether they've been updated.

I'd lean towards making this a required input with no default, so that we have one fewer place to update CUDA versions (both per-repo and the default value).

EOF

- name: Clean up untagged images
uses: snok/container-retention-policy@3b0972b2276b171b212f8c4efbca59ebba26eceb
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
uses: snok/container-retention-policy@3b0972b2276b171b212f8c4efbca59ebba26eceb
uses: snok/container-retention-policy@3b0972b2276b171b212f8c4efbca59ebba26eceb # v3.0.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants