Skip to content

Commit 1938572

Browse files
Fix downstream CI (#432)
Co-authored-by: Pierre Glaser <[email protected]>
1 parent 343da11 commit 1938572

File tree

2 files changed

+35
-11
lines changed

2 files changed

+35
-11
lines changed

.github/scripts/test_downstream_project.sh

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
pushd ../$PROJECT
2-
python -m pytest -vl
2+
echo "${DISABLE_IPV6}"
3+
echo "${PYTEST_ADDOPTS[@]}"
4+
python -m pytest -vl "${PYTEST_ADDOPTS[@]}"
35
TEST_RETURN_CODE=$?
46
popd
57
if [[ "$TEST_RETURN_CODE" != "0" ]]; then

.github/workflows/testing.yml

Lines changed: 32 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -141,9 +141,8 @@ jobs:
141141
if: "contains(github.event.pull_request.labels.*.name, 'ci distributed') || contains(github.event.pull_request.labels.*.name, 'ci downstream')"
142142
env:
143143
PROJECT: distributed
144-
TEST_REQUIREMENTS: pytest pytest-timeout numpy pandas mock bokeh fsspec>=0.3.3
144+
TEST_REQUIREMENTS: cryptography pytest pytest-timeout numpy pandas mock bokeh fsspec>=0.3.3
145145
PROJECT_URL: https://github.com/dask/distributed.git
146-
PYTEST_ADDOPTS: "--timeout-method=thread --timeout=300 -m \"not avoid_travis\" -k \"not test_dask_scheduler and not test_workspace_concurrency and not test_defaults and not test_service_hosts and not test_logging_file_config and not test_hostport and not test_workdir_simple and not test_two_workspaces_in_same_directory and not test_recompute_released_results and not test_connection_args and not test_listen_args and not test_remote_access\""
147146
strategy:
148147
matrix:
149148
python_version: [3.7]
@@ -156,9 +155,22 @@ jobs:
156155
- name: Install project and dependencies
157156
run: |
158157
bash ./.github/scripts/install_downstream_project.sh
158+
- name: Install developed version of cloudpickle
159+
shell: bash
160+
run: |
161+
python -m pip install -e .
159162
- name: Test the downstream project
163+
shell: bash
160164
run: |
161-
bash ./.github/scripts/test_downstream_project.sh
165+
# FIXME ipv6-related failures on Ubuntu github actions CI
166+
# https://github.com/dask/distributed/issues/4514
167+
export DISABLE_IPV6=1
168+
# Skipping
169+
# - test_pickle_empty until https://github.com/dask/distributed/pull/5303 gets merged,
170+
# - test_dont_steal_unknown_function (https://github.com/dask/distributed/issues/3574)
171+
# See https://github.com/cloudpipe/cloudpickle/pull/432
172+
export PYTEST_ADDOPTS=("-k" "not test_pickle_empty and not test_dont_steal_unknown_functions")
173+
source ./.github/scripts/test_downstream_project.sh
162174
163175
joblib-downstream-build:
164176
runs-on: ubuntu-latest
@@ -194,7 +206,7 @@ jobs:
194206
env:
195207
PROJECT: loky
196208
TEST_REQUIREMENTS: "pytest psutil"
197-
PROJECT_URL: https://github.com/tomMoral/loky.git
209+
PROJECT_URL: https://github.com/joblib/loky.git
198210
strategy:
199211
matrix:
200212
python_version: [3.7]
@@ -204,9 +216,13 @@ jobs:
204216
uses: actions/setup-python@v1
205217
with:
206218
python-version: ${{ matrix.python_version }}
207-
- name: Install project and dependencies
219+
- name: Install downstream project and dependencies
208220
run: |
209221
bash ./.github/scripts/install_downstream_project.sh
222+
- name: Install developed version of cloudpickle
223+
shell: bash
224+
run: |
225+
python -m pip install -e .
210226
- name: Test the downstream project
211227
run: |
212228
bash ./.github/scripts/test_downstream_project.sh
@@ -218,21 +234,27 @@ jobs:
218234
PROJECT: ray
219235
strategy:
220236
matrix:
221-
python_version: [3.7]
237+
python_version: [3.8]
222238
steps:
223239
- uses: actions/checkout@v1
224-
- name: Set up Python 3.7
240+
- name: Set up Python 3.8
225241
uses: actions/setup-python@v1
226242
with:
227243
python-version: ${{ matrix.python_version }}
228244
- name: Install project and dependencies
229245
run: |
230246
python -m pip install --upgrade -r dev-requirements.txt
231-
python -m pip install setproctitle psutil ray==0.8.6
247+
python -m pip install setproctitle psutil
248+
# from https://docs.ray.io/en/master/development.html#building-ray
249+
pip install -U https://s3-us-west-2.amazonaws.com/ray-wheels/latest/ray-2.0.0.dev0-cp38-cp38-manylinux2014_x86_64.whl
250+
pushd ..
251+
git clone https://github.com/ray-project/ray.git
252+
pushd ray
253+
python python/ray/setup-dev.py --yes
254+
popd
255+
popd
232256
PROJECT_DIR=$(python -c "import os, ray; print(os.path.dirname(ray.__file__), flush=True)")
233257
rm $PROJECT_DIR/cloudpickle/cloudpickle.py
234-
git clone https://github.com/ray-project/ray.git ../ray
235-
cp -R ../ray/python/ray/tests $PROJECT_DIR/tests
236258
cp cloudpickle/cloudpickle.py $PROJECT_DIR/cloudpickle/cloudpickle.py
237259
cp cloudpickle/compat.py $PROJECT_DIR/cloudpickle/compat.py
238260
cp cloudpickle/cloudpickle_fast.py $PROJECT_DIR/cloudpickle/cloudpickle_fast.py

0 commit comments

Comments
 (0)