Skip to content

Commit

Permalink
best observability
Browse files Browse the repository at this point in the history
  • Loading branch information
C-Loftus committed Dec 10, 2024
1 parent 6c82e0c commit 77d49ed
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 2 deletions.
10 changes: 8 additions & 2 deletions .github/workflows/e2e_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ on:
workflow_dispatch: # Allows manual triggering of the workflow

jobs:
setup-docker-and-run:
setup-docker-and-test:
runs-on: ubuntu-latest

steps:
Expand All @@ -19,4 +19,10 @@ jobs:

# Run the Python script that handles the build and execution
- name: Launch Docker Stack and run tests
run: python3 main.py local && python3 main.py wait_for "http://localhost:9001/minio/health/live" && python3 main.py test
run: python3 main.py local

- name: Wait for minio
run: python3 main.py wait_for "http://localhost:9001/minio/health/live"

- name: Run pytest
run: python3 main.py test
1 change: 1 addition & 0 deletions main.py
Original file line number Diff line number Diff line change
Expand Up @@ -129,6 +129,7 @@ def wait_for_response(url: str):
docker stack does not support conditional waits and the dockerfile logic
would be otherwise messy
"""
print("Waiting for response from " + url)
TIMEOUT_SEC = 180
counter = 0
while True and counter < TIMEOUT_SEC:
Expand Down

0 comments on commit 77d49ed

Please sign in to comment.