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

Add CUDA 11.8 devcontainer to CCCL matrix #173

Merged
merged 1 commit into from
Nov 10, 2023

Conversation

sleeepyjack
Copy link
Contributor

This PR adds a single image to the CCCL devcontainer matrix which is needed to run cuCollections CI tests.

CC @jrhemstad @bdice

@@ -77,6 +77,7 @@ include:
images:
- {features: [*nvhpc_prev, *python, *cccl_dev], env: *nvhpc_env}
- {features: [*nvhpc_curr, *python, *cccl_dev], env: *nvhpc_env}
- {features: [*gcc_11, {<<: *cuda_prev_max, <<: *cccl_cuda_opts}, *python, *cccl_dev], env: *gcc_env}
Copy link
Collaborator

Choose a reason for hiding this comment

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

You put them into the CCCL bracket. Would it make sense to add a separate block with cuCollections as a target.

With that container in here, we are running risk that we might change the devcontainer when we do changes to the CCCL ones

Copy link
Contributor Author

Choose a reason for hiding this comment

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

For cuCollections we try to adhere to the CCCL setup as close as possible. Although, we only test a small subset of the CCCL matrix. I'm fine with leaving this in the CCCL section, since all of the other images in our matrix are CCCL images anyway.

@miscco miscco merged commit a724a0b into rapidsai:branch-23.12 Nov 10, 2023
184 checks passed
@miscco
Copy link
Collaborator

miscco commented Nov 10, 2023

I started a deployment so that the image gets pushed to the registry

https://github.com/rapidsai/devcontainers/actions/runs/6827022148

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.

2 participants