Skip to content

Document postgres POSTGRES_INITDB_XLOGDIR variable #872

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
Apr 6, 2017

Conversation

DanielDent
Copy link
Contributor

@DanielDent DanielDent commented Apr 6, 2017

docker-library/postgres#224 and docker-library/official-images#2762 added the POSTGRES_INITDB_XLOGDIR variable. This adds documentation for that variable.

Copy link
Member

@tianon tianon left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! ❤️ ❤️

(Just one minor note)

@@ -68,6 +68,10 @@ This optional environment variable can be used to define a different name for th

This optional environment variable can be used to send arguments to `postgres initdb`. The value is a space separated string of arguments as `postgres initdb` would expect them. This is useful for adding functionality like data page checksums: `-e POSTGRES_INITDB_ARGS="--data-checksums"`.

### `POSTGRES_XLOG_DIR`
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops, I forgot we changed the name along the way. Updated my PR accordingly.

docker-library/postgres#224 and docker-library/official-images#2762 added the POSTGRES_INITDB_XLOGDIR variable. This adds documentation for that variable.
@DanielDent DanielDent changed the title Document postgres image POSTGRES_XLOG_DIR variable Document postgres POSTGRES_INITDB_XLOGDIR variable Apr 6, 2017
@tianon tianon requested a review from yosifkit April 6, 2017 21:46
@yosifkit yosifkit merged commit 64ccf5e into docker-library:master Apr 6, 2017
@DanielDent DanielDent deleted the patch-1 branch April 6, 2017 22:49
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