Skip to content

Commit

Permalink
fix: Add loki readiness probe to CI
Browse files Browse the repository at this point in the history
  • Loading branch information
lperdereau committed Jul 3, 2023
1 parent 52d17c6 commit c6a2c27
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 3 deletions.
8 changes: 7 additions & 1 deletion .github/workflows/go-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,13 @@ jobs:
image: grafana/loki:2.8.0
ports:
- "3100:3100"
command: -config.file=/etc/loki/local-config.yaml
options: >-
--name=loki1
--health-cmd "curl -f http://localhost:3100/ready"
--health-interval 30s
--health-timeout 10s
--health-retries 5
--health-start-period 30s
steps:

Expand Down
2 changes: 1 addition & 1 deletion pkg/acquisition/modules/loki/loki_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -328,7 +328,7 @@ mode: tail
source: loki
url: http://127.0.0.1:3100
query: >
{server="demo"}
{server="demo"}
`,
expectedErr: "",
streamErr: "",
Expand Down
1 change: 0 additions & 1 deletion test/localstack/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -82,4 +82,3 @@ services:
image: grafana/loki:2.8.0
ports:
- "3100:3100"
command: -config.file=/etc/loki/local-config.yaml

0 comments on commit c6a2c27

Please sign in to comment.