Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change healthcheck postgres #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

patsevanton
Copy link

No description provided.

@dsmyda
Copy link

dsmyda commented Feb 23, 2022

@patsevanton We were actually able to get this to go away without quiet mode by keeping everything in one string

["CMD-SHELL", "pg_isready -d kong -U kong"]

@Vlad412
Copy link

Vlad412 commented Dec 5, 2022

Thank you, it's working

@ayandebnath
Copy link

@patsevanton We were actually able to get this to go away without quiet mode by keeping everything in one string

["CMD-SHELL", "pg_isready -d kong -U kong"]

Solved, thanks.

@@ -9,7 +9,7 @@ services:
- POSTGRES_DB=kong
- POSTGRES_HOST_AUTH_METHOD=trust
healthcheck:
test: ["CMD-SHELL", "pg_isready"]
Copy link

Choose a reason for hiding this comment

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

Might want to reflect this change in the README.md as well.

@duanyrf
Copy link

duanyrf commented May 6, 2023

It worked for me, but I wrote this way:

healthcheck:
      test: pg_isready -d bioteste_dev -U postgres

devraj added a commit to anomaly/lab-python-server that referenced this pull request May 25, 2023
i would say this is a bug in docker compose, putting the healthcheck command in
a single string fixes the issue
peter-evans/docker-compose-healthcheck#17

CLOSES #72
xome added a commit to xome/PenAndPaperDmHelperBackendSpringPostgres that referenced this pull request Jan 10, 2024
@raindongz
Copy link

tried test: ["CMD", "pg_isready", "-q", "-d", "root", "-U", "postgres"] and lot similar approaches and still not work. but this one works perfectly. I am just curious why

Agent-E11 added a commit to Agent-E11/htmx_go that referenced this pull request Feb 10, 2024
@saleh199
Copy link

It worked for me, but I wrote this way:

healthcheck:
      test: pg_isready -d bioteste_dev -U postgres

this works for me, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants