Skip to content

(MODULES-4558) Add support for --data-checksums on initdb #855

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

Closed
wants to merge 1 commit into from

Conversation

atmoz
Copy link

@atmoz atmoz commented Mar 21, 2017

We want to use checksums on data pages. This has to be enabled when running initdb. Currently, the puppetlabs/postgresql module does not seem to support this.

https://tickets.puppetlabs.com/browse/MODULES-4558

@binford2k binford2k changed the title Add support for --data-checksums on initdb (MODULES-4558) Add support for --data-checksums on initdb Mar 21, 2017
@kny78
Copy link

kny78 commented Mar 22, 2017

Could you rebase this into a single commit?

@atmoz
Copy link
Author

atmoz commented Mar 22, 2017

Done. I also added the new option data_checksums to README.md

@mmoll
Copy link
Contributor

mmoll commented Jul 4, 2017

is this the same functionality as #878?

@atmoz
Copy link
Author

atmoz commented Aug 11, 2017

@mmoll Yes

So this PR have been open almost half a year (also used in production), and then this similar PR (#878) gets merged after just 3 days?

Looks like the other PR misses the version check as well.

Come on guys 👎

@mmoll
Copy link
Contributor

mmoll commented Aug 11, 2017

@atmoz could you rebase/rework that PR so the additional check can get in?

@mmoll
Copy link
Contributor

mmoll commented Sep 9, 2017

got in via GH-915, so this PR can get closed

@tphoney
Copy link
Contributor

tphoney commented Sep 28, 2017

Sorry about the slowness. Thanks for your patience @atmoz and apologies.

@tphoney tphoney closed this Sep 28, 2017
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.

4 participants