Skip to content

Commit ce17fbb

Browse files
authored
fix slurm integration test (#949)
* Update dev-requirements.txt, Add docker-compose This fixed the slurm integration test. * Update slurm-local-integration-tests.yaml * Update dev-requirements.txt * Update slurm-local-integration-tests.yaml * Update dev-requirements.txt * Update slurm-local-integration-tests.yaml * Update slurmdockerint.sh * Update slurmdockerint.sh * Update slurm-local-integration-tests.yaml * Update slurmdockerint.sh * Update slurm-local-integration-tests.yaml
1 parent e1eb939 commit ce17fbb

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

.github/workflows/slurm-local-integration-tests.yaml

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ jobs:
3434
set -ex
3535
3636
cd slurm-docker-cluster
37-
docker-compose pull --ignore-pull-failures
37+
docker compose pull --ignore-pull-failures
3838
continue-on-error: true
3939
# cache layers to avoid rebuilding slurm
4040
- uses: satackey/[email protected]
@@ -48,11 +48,10 @@ jobs:
4848
run: |
4949
set -ex
5050
cd slurm-docker-cluster
51-
IMAGE_TAG=$SLURM_VERSION docker-compose up --detach
51+
IMAGE_TAG=$SLURM_VERSION docker compose up --detach
5252
- name: Install Dependencies
53-
run:
53+
run: |
5454
set -ex
55-
5655
pip install wheel
5756
- name: Run Slurm Integration Tests
5857
run: |

dev-requirements.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ torchtext==0.17.1
3131
torchvision==0.17.1
3232
ts==0.5.1
3333
ray[default]
34+
wheel
3435

3536
# lint (linter versions are managed by lintrunner)
3637
lintrunner

0 commit comments

Comments
 (0)