-
Notifications
You must be signed in to change notification settings - Fork 7k
[ci] updating data ci tests to py310 #58378
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
Conversation
Signed-off-by: elliot-barn <[email protected]>
Signed-off-by: elliot-barn <[email protected]>
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.
Bug: Build name mismatch breaks CI dependency references
The data9build step was renamed to data9build-multipy, but the "arrow v9 tests" and "arrow v9 tests (data_non_parallel)" still reference the original data9build name. This occurs in both their depends_on fields and the --build-name argument within their commands, causing CI failures due to missing build steps and artifacts.
.buildkite/data.rayci.yml#L65-L82
Lines 65 to 82 in c1c3d1c
| --worker-id "$${BUILDKITE_PARALLEL_JOB}" --parallelism-per-worker 3 | |
| --build-name data9build | |
| --except-tags data_integration,doctest,data_non_parallel,dask,needs_credentials | |
| depends_on: data9build | |
| - label: ":database: data: arrow v9 tests (data_non_parallel)" | |
| tags: | |
| - data | |
| - data_non_parallel | |
| instance_type: medium | |
| parallelism: 3 | |
| commands: | |
| - bazel run //ci/ray_ci:test_in_docker -- //python/ray/data/... //python/ray/air/... data | |
| --workers "$${BUILDKITE_PARALLEL_JOB_COUNT}" | |
| --worker-id "$${BUILDKITE_PARALLEL_JOB}" | |
| --build-name data9build | |
| --only-tags data_non_parallel | |
| depends_on: data9build |
.buildkite/data.rayci.yml#L8-L16
Lines 8 to 16 in c1c3d1c
| # builds | |
| - name: data9build-multipy | |
| label: "wanda: data9build-py{{matrix}}" | |
| wanda: ci/docker/data9.build.wanda.yaml | |
| matrix: | |
| - "3.10" | |
| env: | |
| PYTHON: "{{matrix}}" | |
| tags: cibase |
upgrading data ci tests to py3.10 postmerge build: https://buildkite.com/ray-project/postmerge/builds/14192 --------- Signed-off-by: elliot-barn <[email protected]> Co-authored-by: Lonnie Liu <[email protected]>
upgrading data ci tests to py3.10 postmerge build: https://buildkite.com/ray-project/postmerge/builds/14192 --------- Signed-off-by: elliot-barn <[email protected]> Co-authored-by: Lonnie Liu <[email protected]>
upgrading data ci tests to py3.10 postmerge build: https://buildkite.com/ray-project/postmerge/builds/14192 --------- Signed-off-by: elliot-barn <[email protected]> Co-authored-by: Lonnie Liu <[email protected]> Signed-off-by: Aydin Abiar <[email protected]>
upgrading data ci tests to py3.10 postmerge build: https://buildkite.com/ray-project/postmerge/builds/14192 --------- Signed-off-by: elliot-barn <[email protected]> Co-authored-by: Lonnie Liu <[email protected]> Signed-off-by: YK <[email protected]>
upgrading data ci tests to py3.10 postmerge build: https://buildkite.com/ray-project/postmerge/builds/14192 --------- Signed-off-by: elliot-barn <[email protected]> Co-authored-by: Lonnie Liu <[email protected]>
upgrading data ci tests to py3.10
postmerge build: https://buildkite.com/ray-project/postmerge/builds/14192