Skip to content

Commit

Permalink
see if I can get the step to fail if there is an error
Browse files Browse the repository at this point in the history
  • Loading branch information
mikemhenry committed Jul 25, 2023
1 parent 7724069 commit 37c5dd9
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions .github/workflows/docker.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ name: Create and publish a Docker image
on:
push:
branches:
- feat/improve_docker_building
- feat/improve_docker_building
workflow_dispatch:

defaults:
Expand Down Expand Up @@ -101,7 +101,7 @@ jobs:
echo "${{ secrets.OE_LICENSE }}" > oe_license_dir/oe_license.txt
docker run --rm ${{ steps.fqirp.outputs.FQIRP }} perses-cli --help
# Mount oe_license_dir (which should have the license file) to /openeye on container for testing
#docker run -e GITHUB_ACTIONS=true --mount type=bind,source=$PWD/oe_license_dir/,target=/openeye/,readonly --rm ${{ steps.fqirp.outputs.FQIRP }} pytest --pyargs perses -a "not advanced" -n auto -m "not gpu_needed" -v
docker run -e GITHUB_ACTIONS=true --mount type=bind,source=$PWD/oe_license_dir/,target=/openeye/,readonly --rm ${{ steps.fqirp.outputs.FQIRP }} pytest --pyargs perses -a "not advanced" -n auto -m "not gpu_needed" -v
- name: Push Docker image
uses: docker/build-push-action@ad44023a93711e3deb337508980b4b5e9bcdc5dc
Expand All @@ -124,6 +124,7 @@ jobs:

- name: Test & Push Apptainer Image
run: |
set -e
mkdir test_apptainer
cd test_apptainer
singularity run ../perses_${{ steps.latest-version.outputs.VERSION }}.sif perses-cli --help
Expand Down

0 comments on commit 37c5dd9

Please sign in to comment.