-
Notifications
You must be signed in to change notification settings - Fork 888
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 #17013
base: branch-24.12
Are you sure you want to change the base?
Conversation
@@ -5,6 +5,8 @@ set -euo pipefail | |||
|
|||
. /opt/conda/etc/profile.d/conda.sh | |||
|
|||
RAPIDS_VERSION_MAJOR_MINOR="$(rapids-version-major-minor)" |
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.
In build_docs.sh
we export
this, but here we're just setting it locally. I doubt it matters here but it might be worth establishing consistent patterns. Also applies to other files.
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.
My thinking here was "export it if it's already exported (as in the docs example), otherwise make it local", since I thought that had the least risk of causing other surprising problems.
I clicked around a little, and I think:
RAPIDS_VERSION
= does not even need to be set in docs-build scriptsRAPIDS_VERSION_MAJOR_MINOR
= does not need to be exportedRAPIDS_VERSION_NUMBER
= needs to be exported, forrapids-upload-docs
(code link in gha-tools)
For all the other scripts that are not docs builds, this PR only introduces NEW variables, so I think not exporting them is slightly safer.
Just pushed 16367a5 doing that. If that works, I'll make similar changes in the other PRs.
Description
Contributes to rapidsai/build-planning#106
Proposes specifying the RAPIDS version in
conda install
calls in CI that install CI artifacts, to reduce the risk of CI jobs picking up artifacts from other releases.Checklist