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

Fix creation of recovery.conf file when recovery configuration is not specified #995

Merged
merged 3 commits into from
Jun 19, 2018

Conversation

cdloh
Copy link

@cdloh cdloh commented Jun 7, 2018

Fixes https://tickets.puppetlabs.com/browse/MODULES-4276

See also #603 (comment)

Rebase of orginial MR #834

@cdloh cdloh force-pushed the fix-recovery-conf branch 3 times, most recently from 6ceb484 to 1f92473 Compare June 7, 2018 11:18
Copy link
Member

@david22swan david22swan left a comment

Choose a reason for hiding this comment

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

LGTM but would like a second plus one

Copy link
Member

@david22swan david22swan left a comment

Choose a reason for hiding this comment

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

Please add docs to reflect your changes

@david22swan
Copy link
Member

@david22swan
Copy link
Member

Passed on Adhoc

@cdloh
Copy link
Author

cdloh commented Jun 19, 2018

@david22swan in my opinion the doco already suggests this behavior. Is there somewhere else that I need to update? See below snippet from the repo readme

Only the specified parameters are recognized in the template. The recovery.conf is only be created if at least one parameter is set and manage_recovery_conf is set to true.

@cdloh
Copy link
Author

cdloh commented Jun 19, 2018

I've added further clarification within the readme in the Defined Types area of the documentation

@david22swan david22swan merged commit b9a27d5 into puppetlabs:master Jun 19, 2018
cegeka-jenkins pushed a commit to cegeka/puppet-postgresql that referenced this pull request Feb 3, 2022
Fix creation of recovery.conf file when recovery configuration is not specified
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants