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

(maint) Make PDB container wait longer on Postgres #56

Conversation

Iristyle
Copy link
Contributor

@Iristyle Iristyle commented Apr 13, 2019

This is another approach to the PR against PDB at puppetlabs/puppetdb#2904

  • PDB has a configurable wait time on Postgres when it is starting up, which
    defaults to 90 seconds

    This doesn't appear to be long enough in Azure based on failures like:

   wtfc.sh: waiting 90 seconds for nc -v -w 1 -z 'postgres' 5432
   .........................................................................................(/docker-entrypoint.d/20-wait-for-hosts.sh) Error: host postgres:5432 does not appear to be listening
   wtfc.sh: timeout occurred after waiting 90 seconds for nc -v -w 1 -z 'postgres' 5432 to return status: 0 (was status: 143)

 - PDB has a configurable wait time on Postgres when it is starting up, which
   defaults to 90 seconds

   This doesn't appear to be long enough in Azure based on failures like:

   wtfc.sh: waiting 90 seconds for nc -v -w 1 -z 'postgres' 5432
   .........................................................................................(/docker-entrypoint.d/20-wait-for-hosts.sh) Error: host postgres:5432 does not appear to be listening
   wtfc.sh: timeout occurred after waiting 90 seconds for nc -v -w 1 -z 'postgres' 5432 to return status: 0 (was status: 143)
@Iristyle
Copy link
Contributor Author

Quite certain the problem here was not the wait time, but rather interesting DNS resolution problems, ultimately fixed in #61

@Iristyle Iristyle closed this May 16, 2019
Iristyle added a commit that referenced this pull request Apr 9, 2021
(maint) Update README with external Postgres info / external CA info
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.

1 participant