Skip to content

Commit

Permalink
Removing sleep -> relying on healthchecks
Browse files Browse the repository at this point in the history
  • Loading branch information
meetagrawal09 authored Sep 14, 2024
1 parent 5896395 commit c6d38df
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions .github/workflows/docker-stack-sipnet.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ jobs:
cp docker-compose.dev.yml docker-compose.override.yml
sed -i'' '/R_library/d' docker-compose.override.yml
- name: Build Containers #sleep should be removed once we merge #3354
- name: Build Containers
run: |
cp docker/env.example .env
echo "COMPOSE_PROJECT_NAME=pecan" >> .env
Expand All @@ -49,7 +49,6 @@ jobs:
echo "GID=$(id -g)" >> .env
docker compose up --wait -d postgres
sleep 30
docker run --rm --network pecan_pecan pecan/db:ci
docker run --rm --network pecan_pecan --volume pecan_pecan:/data --env FQDN=docker pecan/data:develop
docker image rm pecan/db:ci pecan/data:develop
Expand All @@ -58,10 +57,9 @@ jobs:
run: |
docker compose up --wait -d sipnet executor
- name: Check if containers are running #should be removed once we merge #3354
- name: List running containers
run: |
docker ps
sleep 50
- name: Run SIPNET
run: |
Expand Down

0 comments on commit c6d38df

Please sign in to comment.