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

Run restart command right away on the postgresql service. #251

Merged
merged 1 commit into from
May 22, 2015

Conversation

rbclark
Copy link
Contributor

@rbclark rbclark commented Apr 29, 2015

This fixes errors when using docker, without it the error psql: could not connect to server: no such file or directory is displayed when this recipe is used with docker and chef-solo. This is the same issue that is affecting #206 (comment) however I feel firing 1 restart is better than firing 1 for each configuration change.

It seems to me that the issue here is most likely that the server is started by default however does not have the proper configuration and thus connecting fails. This should not break any existing functionality.

This fixes errors when using docker, without it the error `psql: could not connect to server: no such file or directory` is displayed when this recipe is used with docker and chef-solo. This is the same issue that is affecting sous-chefs#206 (comment) however I feel firing 1 restart is better than firing 1 for each configuration change.
slyness pushed a commit that referenced this pull request May 22, 2015
Run restart command right away on the postgresql service.
@slyness slyness merged commit 0acfa99 into sous-chefs:develop May 22, 2015
chrisroberts added a commit that referenced this pull request May 28, 2015
@lock
Copy link

lock bot commented Jul 24, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Jul 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants