Skip to content

Commit

Permalink
Merge pull request #2038 from freedomofpress/rm-dockerize
Browse files Browse the repository at this point in the history
Use curl instead of dockerize in SDK CI
  • Loading branch information
zenmonkeykstop authored May 29, 2024
2 parents 5defc4f + 4af8d07 commit 86b1d07
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions .github/workflows/sdk.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,16 +30,14 @@ jobs:
- name: Install dependencies
run: |
pip install poetry==1.6.1
wget https://github.com/jwilder/dockerize/releases/download/$DOCKERIZE_VERSION/dockerize-linux-amd64-$DOCKERIZE_VERSION.tar.gz &&
tar -C /usr/local/bin -xzvf dockerize-linux-amd64-$DOCKERIZE_VERSION.tar.gz &&
rm dockerize-linux-amd64-$DOCKERIZE_VERSION.tar.gz
- name: Run tests without VCR cassettes
run: |
# Start the server in the background
NUM_SOURCES=5 make -C securedrop-server dev &
# And install deps
poetry -C client install --no-ansi
# Wait for server to come up
dockerize -wait http://127.0.0.1:8080 -timeout 5m
echo "Polling with curl --silent until the server is up..."
while ! curl --silent --fail --output /dev/null http://127.0.0.1:8080; do sleep 5; done
# Run tests
make -C client regenerate-sdk-cassettes

0 comments on commit 86b1d07

Please sign in to comment.