Skip to content

Commit

Permalink
Move checking the webapp so we dump the docker logs on failure
Browse files Browse the repository at this point in the history
  • Loading branch information
martinburchell committed Aug 2, 2024
1 parent d2efdfa commit 4156e9b
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 8 deletions.
9 changes: 5 additions & 4 deletions .github/workflows/biotestmine.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,11 @@ jobs:
# Wait 15 minutes
wait-for-it localhost:9999 --timeout=900
- name: Check webapp
run: |
set -eux -o pipefail
curl -I -L --retry 10 --fail --insecure "http://localhost:9999/biotestmine/"
- name: Dump Docker logs
if: failure()
run: |
Expand All @@ -56,7 +61,3 @@ jobs:
DOCKER_UID=$(id -u) DOCKER_GID=$(id -g) docker compose -f local.docker-compose.yml logs postgres
DOCKER_UID=$(id -u) DOCKER_GID=$(id -g) docker compose -f local.docker-compose.yml logs solr
DOCKER_UID=$(id -u) DOCKER_GID=$(id -g) docker compose -f local.docker-compose.yml logs tomcat
- name: Check webapp
run: |
set -eux -o pipefail
curl -I -L --retry 10 --fail --insecure "http://localhost:9999/biotestmine/"
9 changes: 5 additions & 4 deletions .github/workflows/camchildmine.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,11 @@ jobs:
# Wait 15 minutes
wait-for-it localhost:9999 --timeout=900
- name: Check webapp
run: |
set -eux -o pipefail
curl -I -L --retry 10 --fail --insecure "http://localhost:9999/camCHILDMine/"
- name: Dump Docker logs
if: failure()
run: |
Expand All @@ -61,7 +66,3 @@ jobs:
DOCKER_UID=$(id -u) DOCKER_GID=$(id -g) docker compose -f local.docker-compose.yml logs postgres
DOCKER_UID=$(id -u) DOCKER_GID=$(id -g) docker compose -f local.docker-compose.yml logs solr
DOCKER_UID=$(id -u) DOCKER_GID=$(id -g) docker compose -f local.docker-compose.yml logs tomcat
- name: Check webapp
run: |
set -eux -o pipefail
curl -I -L --retry 10 --fail --insecure "http://localhost:9999/camCHILDMine/"

0 comments on commit 4156e9b

Please sign in to comment.