Skip to content

Commit

Permalink
[ci] No early kickoff by default for workflow test (ray-project#34213)
Browse files Browse the repository at this point in the history
Signed-off-by: rickyyx <[email protected]>

We have seen workflow test fails on PRs with totally unrelated content because of the re-using of the cached docker image.
- ray-project#34101

Seems the workflow test does have a dependency on the wheels built.

Signed-off-by: elliottower <[email protected]>
  • Loading branch information
rickyyx authored and elliottower committed Apr 22, 2023
1 parent b480937 commit d49858e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .buildkite/pipeline.ml.yml
Original file line number Diff line number Diff line change
Expand Up @@ -478,7 +478,7 @@
- ./ci/run/run_bazel_test_with_sharding.sh --config=ci $(./ci/run/bazel_export_options) --build_tests_only --test_tag_filters=ray_data python/ray/air/...

- label: "Workflow tests"
conditions: ["NO_WHEELS_REQUIRED", "RAY_CI_PYTHON_AFFECTED", "RAY_CI_WORKFLOW_AFFECTED"]
conditions: ["RAY_CI_PYTHON_AFFECTED", "RAY_CI_WORKFLOW_AFFECTED"]
instance_size: medium
commands:
- cleanup() { if [ "${BUILDKITE_PULL_REQUEST}" = "false" ]; then ./ci/build/upload_build_info.sh; fi }; trap cleanup EXIT
Expand Down

0 comments on commit d49858e

Please sign in to comment.