Skip to content

Add documentation for new postgres initdb logic #291

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

Merged
merged 1 commit into from
Jul 24, 2015

Conversation

tianon
Copy link
Member

@tianon tianon commented Jul 23, 2015

Documentation change for docker-library/postgres#75.

@yosifkit
Copy link
Member

LGTM

yosifkit added a commit that referenced this pull request Jul 24, 2015
Add documentation for new postgres initdb logic
@yosifkit yosifkit merged commit 7bc4ab6 into docker-library:master Jul 24, 2015
@yosifkit yosifkit deleted the postgres-initdb branch July 24, 2015 17:59
@md5
Copy link
Contributor

md5 commented Jul 24, 2015

It might be good to point out that psql commands should generally be run as user postgres and that no password is necessary owing to the use of trust authentication for a local Unix socket connection.

@tianon
Copy link
Member Author

tianon commented Aug 21, 2015 via email

@md5
Copy link
Contributor

md5 commented Aug 22, 2015

I could do that. However, I think that the issue of whether *.sql files should be run as postgres or $POSTGRES_USER should probably be resolved first, since both of those could probably be documented at the same time.

@md5
Copy link
Contributor

md5 commented Aug 22, 2015

cf. docker-library/postgres#81

@tianon
Copy link
Member Author

tianon commented Aug 22, 2015 via email

@md5
Copy link
Contributor

md5 commented Aug 22, 2015

See docker-library/postgres#82 for the entrypoint update and #319 for the docs update.

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.

3 participants