[CI] Add filter_release_build_yaml to trim build steps by test needs#62292
[CI] Add filter_release_build_yaml to trim build steps by test needs#62292andrew-anyscale wants to merge 1 commit intoandrew/revup/master/array-releasefrom
Conversation
|
Reviews in this chain: |
|
There was a problem hiding this comment.
Code Review
This pull request adds functionality to filter build.rayci.yml files, retaining only the Python versions, image types, and CUDA platforms required by selected tests. It introduces several helper functions and comprehensive unit tests. The review feedback suggests optimizing the filtering logic by using the _filter_array_dimension helper and removing a redundant conditional check.
50071b5 to
a09ab42
Compare
b3d5a18 to
6786502
Compare
a09ab42 to
c6e4a1d
Compare
fea08d8 to
95a8451
Compare
c6e4a1d to
64db7af
Compare
95a8451 to
4fbf70f
Compare
64db7af to
0be9745
Compare
4fbf70f to
6a82f61
Compare
0aab2cd to
b2a71ba
Compare
6a82f61 to
a1adc68
Compare
18fd7b0 to
b1d502a
Compare
a1adc68 to
576a6ff
Compare
576a6ff to
4e8ebb1
Compare
b1d502a to
230d454
Compare
Add collect_needed_variants() to extract python versions, image types, and CUDA platforms from selected tests. Add filter_release_build_yaml() to rewrite build.rayci.yml in-place, removing steps for unneeded image types and trimming array dimensions (python, platform, cuda) and adjustments to only needed values. Steps left with no viable combinations are removed entirely. This is the library/logic layer — the init script integration comes in the next commit. Topic: release-build-filter Relative: array-release Labels: draft Signed-off-by: andrew <andrew@anyscale.com>
4e8ebb1 to
3bdbf6e
Compare
230d454 to
920c5eb
Compare
Add collect_needed_variants() to extract python versions, image types, and CUDA platforms from selected tests. Add filter_release_build_yaml() to rewrite build.rayci.yml in-place, removing steps for unneeded image types and trimming array dimensions (python, platform, cuda) and adjustments to only needed values. Steps left with no viable combinations are removed entirely.
This is the library/logic layer — the init script integration comes in the next commit.
Topic: release-build-filter
Relative: array-release
Labels: draft
Signed-off-by: andrew andrew@anyscale.com