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

make conda installs in CI stricter #6103

Merged
merged 3 commits into from
Oct 10, 2024

Conversation

jameslamb
Copy link
Member

@jameslamb jameslamb commented Oct 8, 2024

Description

Contributes to rapidsai/build-planning#106

Proposes specifying the RAPIDS version in conda install calls that install CI artifacts, to reduce the risk of CI jobs picking up artifacts from other releases.

Notes for Reviewers

This only changes the docs build, because other packages already solve in a single conda install with version constraints, thanks to #5781.

@jameslamb jameslamb added 2 - In Progress Currenty a work in progress improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Oct 8, 2024
@github-actions github-actions bot added the ci label Oct 8, 2024
@jameslamb jameslamb changed the title WIP: make conda installs in CI stricter make conda installs in CI stricter Oct 8, 2024
@jameslamb jameslamb added 3 - Ready for Review Ready for review by team and removed 2 - In Progress Currenty a work in progress labels Oct 8, 2024
@jameslamb jameslamb marked this pull request as ready for review October 8, 2024 15:05
@jameslamb jameslamb requested a review from a team as a code owner October 8, 2024 15:05
@jameslamb jameslamb requested a review from bdice October 8, 2024 15:05
@jameslamb jameslamb removed the 3 - Ready for Review Ready for review by team label Oct 10, 2024
@dantegd
Copy link
Member

dantegd commented Oct 10, 2024

/merge

@rapids-bot rapids-bot bot merged commit 22342aa into rapidsai:branch-24.12 Oct 10, 2024
63 checks passed
@jameslamb jameslamb deleted the ci-conda-installs branch October 10, 2024 15:36
@jameslamb
Copy link
Member Author

Thanks for merging this @dantegd . Sorry, I realized there was a minor mistake here and hadn't yet made it back to update this PR. See #6104.

@dantegd
Copy link
Member

dantegd commented Oct 10, 2024

Apologies for merging too soon @jameslamb, saw Bradley's approval and was happy to get it in asap :)

@jameslamb
Copy link
Member Author

No prob, my fault for not making it back here fast enough 😂

rapids-bot bot pushed a commit that referenced this pull request Oct 11, 2024
Follow-up to #6103.

In that PR, I'd removed the `export RAPIDS_MAJOR_MINOR_VERSION`. I realized that this project's Doxygen setup actually expects that to be set in the environment.

https://github.com/rapidsai/cuml/blob/22342aad0ecd327e3140471c479d01d83b932c23/ci/build_docs.sh#L35

https://github.com/rapidsai/cuml/blob/22342aad0ecd327e3140471c479d01d83b932c23/cpp/Doxyfile.in#L41

This fixes that, sorry 😬

Authors:
  - James Lamb (https://github.com/jameslamb)

Approvers:
  - https://github.com/jakirkham
  - Mike Sarahan (https://github.com/msarahan)

URL: #6104
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants