Skip package tests during Python Conda build - #752
Open
msarahan wants to merge 1 commit into
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Posted by Codex (GPT-5) on Michael Sarahan’s behalf. Treat this message as LLM-generated.
Why
The Python Conda recipe builds
distributed-ucxx, whose runtime requirements include the train-versionedrapids-dask-dependency. Rattler currently tests each newly built output as part of the build invocation. Setting up thedistributed-ucxxpackage-test environment therefore requires a matchingrapids-dask-dependencypackage before the rest of that RAPIDS train is available. This makes artifact production fail even thoughrapids-dask-dependencyis not required to compile or package UCXX.The package-install and import checks belong in UCXX’s separate test jobs, where runtime packages are available. Keeping them out of the artifact-build invocation also avoids adding a false build-time dependency from UCXX to the repository that produces
rapids-dask-dependency.Change
Pass
--test skipto the rattler invocation for the Python Conda recipe. The C++ Conda recipe is unchanged.This is deliberately scoped to UCXX. A candidate-build-wide policy may replace this repository-specific setting once the candidate build system is used for these builds.
Validation
bash -n ci/build_python.shpre-commit run --all-filesverify-alpha-spechook was rerun with network access after its sandboxed attempt could not resolve GitHub; it passed.