Skip to content
This repository has been archived by the owner on Jul 25, 2022. It is now read-only.

Update influxdb healthcheck #1986

Merged
merged 1 commit into from
Jun 18, 2020
Merged

Update influxdb healthcheck #1986

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions docker/docker-compose.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,9 +33,9 @@ services:
- influxdb:/var/lib/influxdb/
- "manager-config:/var/lib/chroma"
healthcheck:
test: ["CMD-SHELL", "influx -host influxdb -execute 'show databases'"]
test: ["CMD-SHELL", "curl --fail influxdb:8086/ping?wait_for_leader=10s"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can also just use influx -host influxdb -execute 'exit' instead. It's what's used during setup to wait for influx to be online.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like that takes ~ 50% of a core:

Screen Shot 2020-06-18 at 11 08 10 AM

Keep in mind this is being run every 5s.

interval: 5s
timeout: 5s
timeout: 15s
retries: 5
start_period: 60s
ports:
Expand Down