Skip to content

Allow reuse of resolved Python test matrix snapshots - #656

Closed
vyasr wants to merge 2 commits into
mainfrom
codex/resolved-python-test-matrices
Closed

vyasr wants to merge 2 commits into
mainfrom
codex/resolved-python-test-matrices

Conversation

@vyasr

@vyasr vyasr commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Description

Add a reusable resolver that returns wheel-test and Conda Python-test matrices from one preparation of the shared matrix definitions. Callers can plan complementary coverage using a single snapshot, without copying these definitions into downstream repositories.

The wheel and Conda Python test workflows accept an optional resolved-matrix JSON input. Matrix validation, existing filters, and pure-package selectors remain centralized in the shared implementation. When no snapshot is supplied, callers retain the existing resolution behavior. Supplied snapshots do not reread mutable matrix definitions.

Regression tests cover default callers, filtering, pure-package selectors, duplicate matrix types, malformed inputs, and reuse despite changed source definitions. The resolver and consumer should use the same shared-workflows revision.

Pin the existing proxy-cache actions to their current revision to satisfy the repository's pinned security checks on the affected workflows.

@vyasr
vyasr requested a review from a team as a code owner October 8, 2026 20:02
@vyasr vyasr added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Oct 8, 2026
@vyasr
vyasr requested review from KyleFromNVIDIA and removed request for a team October 8, 2026 20:02
@vyasr
vyasr marked this pull request as draft October 8, 2026 20:47
@vyasr

vyasr commented Oct 8, 2026

Copy link
Copy Markdown
Contributor Author

Closing in favor of implementing the complementary CUDA coverage policy entirely in cuDF using the existing compute-matrix and matrix_filter interfaces. We accept independently resolved matrix snapshots within a workflow run, so no shared-workflows API changes are needed.

@vyasr vyasr closed this Oct 8, 2026
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.

1 participant