Skip to content

Commit

Permalink
Add slurm Ci test back (#46)
Browse files Browse the repository at this point in the history
  • Loading branch information
danielholanda authored Dec 1, 2023
1 parent c55c6dc commit e77ac15
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 14 deletions.
26 changes: 12 additions & 14 deletions .github/workflows/test_turnkey.yml
Original file line number Diff line number Diff line change
Expand Up @@ -106,21 +106,19 @@ jobs:
sudo service slurmd start
sudo service slurmctld start
sudo service munge start
# TODO: Slurm test is commented out because it isn't working in OMZ CI
# @Daniel to fix and un-comment
# - name: Test turnkey on Slurm
# if: runner.os != 'Windows'
# shell: bash -el {0}
# run: |
# # Create conda environment for Slurm using srun (sbatch + wait)
# export SKIP_REQUIREMENTS_INSTALL="True"
# export TORCH_CPU="True"
# srun src/turnkeyml/cli/setup_venv.sh
- name: Test turnkey on Slurm
if: runner.os != 'Windows'
shell: bash -el {0}
run: |
# Create conda environment for Slurm using srun (sbatch + wait)
export SKIP_REQUIREMENTS_INSTALL="True"
export TORCH_CPU="True"
srun src/turnkeyml/cli/setup_venv.sh
# # Run tests on Slurm
# export TURNKEY_SLURM_USE_DEFAULT_MEMORY="True"
# turnkey benchmark models/selftest/linear.py --build-only --use-slurm --cache-dir local_cache
# bash test/helpers/check_slurm_output.sh slurm-2.out
# Run tests on Slurm
export TURNKEY_SLURM_USE_DEFAULT_MEMORY="True"
turnkey benchmark models/selftest/linear.py --build-only --use-slurm --cache-dir local_cache
bash test/helpers/check_slurm_output.sh slurm-2.out
# Below tests are commented out as the GitHub runner runs out of space installing the requirements
# - name: Check installation of requirements.txt and their compatibility with turnkey
Expand Down
Empty file modified src/turnkeyml/cli/setup_venv.sh
100644 → 100755
Empty file.

0 comments on commit e77ac15

Please sign in to comment.