Skip to content

Commit

Permalink
service can't have arguments, going back to docker run
Browse files Browse the repository at this point in the history
  • Loading branch information
zschumacher committed Jan 30, 2025
1 parent cf6a193 commit 2097c86
Showing 1 changed file with 1 addition and 14 deletions.
15 changes: 1 addition & 14 deletions .github/workflows/test-pinot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,18 +18,6 @@ jobs:
matrix:
python-version: [ '3.9', '3.10', '3.11', '3.12', '3.13' ]

services:

pinot:
image: apachepinot/pinot:latest
ports:
- 9000:9000
- 8099:8000
options: >- # oracle has a longer startup so give the healthcheck more retries
--health-cmd "curl -f http://localhost:9000/health || exit 1"
--health-interval 10s
--health-timeout 5s
--health-retries 10

steps:
- name: checkout
Expand Down Expand Up @@ -62,8 +50,7 @@ jobs:
env:
PYTHON_VERSION: ${{ matrix.python-version }}
run: |
make waitforpinot
make pinottables
make pinot
poetry run pytest -m "pinot" --cov=. --cov-branch -v --durations=25 --cov-report=xml
- uses: codecov/codecov-action@v5
Expand Down

0 comments on commit 2097c86

Please sign in to comment.