Skip to content

Commit

Permalink
[ci] fix pip-compile job (ray-project#42321)
Browse files Browse the repository at this point in the history
Move pip-compile from forge to one of the ci build so it can run on the same python version with everywhere else. Also move it to others and make it logs verbose.

Signed-off-by: can <can@anyscale.com>
  • Loading branch information
can-anyscale authored Jan 11, 2024
1 parent b567d9f commit 91f312e
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 11 deletions.
10 changes: 0 additions & 10 deletions .buildkite/build.rayci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -150,13 +150,3 @@ steps:
- "3.8"
- "3.9"
- "3.10"

- label: ":tapioca: build: pip-compile dependencies"
instance_type: small
commands:
- rm ./python/requirements_compiled.txt
- ./ci/ci.sh compile_pip_dependencies
- cp -f ./python/requirements_compiled.txt /artifact-mount/
soft_fail: true
depends_on:
- forge
12 changes: 12 additions & 0 deletions .buildkite/others.rayci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,18 @@ steps:
- name: doctestbuild
wanda: ci/docker/doctest.build.wanda.yaml

- label: ":tapioca: build: pip-compile dependencies"
instance_type: small
commands:
- rm ./python/requirements_compiled.txt
- ./ci/ci.sh compile_pip_dependencies
- cp -f ./python/requirements_compiled.txt /artifact-mount/
soft_fail: true
job_env: oss-ci-base_build
depends_on:
- forge
- oss-ci-base_build

# test
- label: doc tests
instance_type: large
Expand Down
2 changes: 1 addition & 1 deletion ci/ci.sh
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ compile_pip_dependencies() {
python -c "import torch" 2>/dev/null && HAS_TORCH=1
pip install --no-cache-dir numpy torch

pip-compile --resolver=backtracking -q \
pip-compile --verbose --resolver=backtracking \
--pip-args --no-deps --strip-extras --no-header -o \
"${WORKSPACE_DIR}/python/$TARGET" \
"${WORKSPACE_DIR}/python/requirements.txt" \
Expand Down

0 comments on commit 91f312e

Please sign in to comment.