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

few postgres defaults changed in postgres 10, which are not yet reflected in defaults/main.yml #484

Closed
neiaberau opened this issue May 31, 2020 · 2 comments

Comments

@neiaberau
Copy link

neiaberau commented May 31, 2020

(this is a more general issue than the previous one or overlapping:)
it seems, between version 9 and 10 some defaults changed, but got never adapted in defaults/main.yml, was this on purpose as for most installations not relevant or missing?

example: max_wal_senders or wal_level:

defaults/main.yml:postgresql_max_wal_senders: 0
templates/postgresql.conf-10.orig:#max_wal_senders = 10 # max number of walsender processes
templates/postgresql.conf-11.orig:#max_wal_senders = 10 # max number of walsender processes
templates/postgresql.conf-12.orig:#max_wal_senders = 10 # max number of walsender processes
templates/postgresql.conf-9.5.orig:#max_wal_senders = 0 # max number of walsender processes
templates/postgresql.conf-9.6.orig:#max_wal_senders = 0 # max number of walsender processes

defaults/main.yml:postgresql_wal_level: "minimal" # minimal, archive (<= 9.5), hot_standby (<= 9.5), replica (>= 9.6), or logical
templates/postgresql.conf-10.orig:#wal_level = replica # minimal, replica, or logical
templates/postgresql.conf-11.orig:#wal_level = replica # minimal, replica, or logical
templates/postgresql.conf-12.orig:#wal_level = replica # minimal, replica, or logical
templates/postgresql.conf-9.5.orig:#wal_level = minimal # minimal, archive, hot_standby, or logical
templates/postgresql.conf-9.6.orig:#wal_level = minimal # minimal, replica, or logical

@neiaberau neiaberau changed the title postgres defaults change in postgres 10, which are not in defaults/main.yml few postgres defaults changed in postgres 10, which are not yet reflected in defaults/main.yml May 31, 2020
gclough added a commit to gclough/postgresql that referenced this issue Mar 11, 2021
@gclough
Copy link
Collaborator

gclough commented Mar 11, 2021

Thanks for the report @neiaberau. I have rolled these updates into #504 , so that PostgreSQL v13 (and the others you've listed) will now get the correct defaults.

defaults/main.yml

postgresql_max_wal_senders: 10
postgresql_wal_level: "replica"  # minimal, archive (<= 9.5), hot_standby (<= 9.5), replica (>= 9.6), or logical

vars/postgresql_9.5.yml

postgresql_max_wal_senders: 0
postgresql_wal_level: "minimal"

vars/postgresql_9.6.yml

postgresql_max_wal_senders: 0
postgresql_wal_level: "minimal"

That should merge soon, but please review and reopen this if you don't think it's been resolved.

@gclough gclough closed this as completed Mar 11, 2021
@halvaborsch
Copy link

I don't get it what was happened in this commit. How do i can change postgresql_wal_level value in my playbook group_vars or inventory vars now? My variables will be always override with include_vars that has greater priority in ansible variable precedence.

Or i misunderstand something?

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

No branches or pull requests

3 participants