Skip to content

Update Patroni config file #551

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: release-17.5.3
Choose a base branch
from
Open

Update Patroni config file #551

wants to merge 2 commits into from

Conversation

Andriciuc
Copy link
Collaborator

The original config file was taken from here and it is currently replaced with a more "up to date" version here.

@Andriciuc Andriciuc requested a review from nastena1606 as a code owner August 26, 2025 12:20
@Andriciuc Andriciuc added the documentation Improvements or additions to documentation label Aug 26, 2025
@Andriciuc
Copy link
Collaborator Author

Added:

parameters:
        unix_socket_directories: /tmp
        # Use unix_socket_directories: /var/run/postgresql for Debian/Ubuntu distributions

jeltz
jeltz requested changes Aug 26, 2025
View reviewed changes
Copy link
Collaborator

@jeltz jeltz left a comment

Choose a reason for hiding this comment

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

@jobinau do you have any comments on this config?

contrib/pg_tde/documentation/docs/index/tde-limitations.md
name: postgresql0

scope: tde
name: {{ansible_host}}
Copy link
Collaborator

Choose a reason for hiding this comment

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

These are ansible specific stuff which I think we should remove, maybe replace it with pg1 as in Joni's example?

contrib/pg_tde/documentation/docs/index/tde-limitations.md
authentication:
replication:
username: replicator
password: rep-pass
password: {{secret}}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Again more ansible/jinja2 templating but here I am less sure what to replace it with.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jeltz jeltz jeltz requested changes

@nastena1606 nastena1606 Awaiting requested review from nastena1606 nastena1606 is a code owner

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Andriciuc @jeltz