Skip to content

Patroni related standard template improvements suggested : PG-1529 #779

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

Open
wants to merge 2 commits into
base: 17
Choose a base branch
from

Conversation

jobinau
Copy link
Contributor

@jobinau jobinau commented Apr 4, 2025

Majorly 5 changes

  1. pg_hba entries are moved to DCS section. This may simplify the DBA's work.
  2. md5 is converted to scram-sha-256. Because PG18 is deprecating md5.
  3. Sample recovery_conf. Because majority of deployement needs it.
  4. Section for creating additional users are removed. Additional users can be created at anytime.
  5. Sample watchdog section . Integration of Linux Watchdog is highly encouraged to perform STONITH / fencing when node hangs.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
1.  entries are moved to DCS section. This may simplify the DBA's work.
2.  is converted to . Because PG18 is deprecating md5.
3. Sample . Because majority of deployement needs it.
4. Section for creating additional users are removed. Additional users can be created at anytime.
5. Sample  section . Integration of Linux Watchdog is highly encouraged to perform STONITH / fencing when node hangs.
@jobinau
Copy link
Contributor Author

jobinau commented Apr 4, 2025

@jobinau jobinau changed the title Majorly 5 changes Patroni related standard template improvements suggested : PG-1529 Apr 4, 2025
@ImTheKai ImTheKai requested review from nastena1606 and elchinoo April 4, 2025 15:32
@ImTheKai
Copy link
Contributor

ImTheKai commented Apr 4, 2025

This change would also be needed for all other PostgreSQL branches, right?

@nastena1606
Copy link
Contributor

Why don't we have a watchdog section for Yum setup?

@jobinau
Copy link
Contributor Author

jobinau commented Apr 8, 2025

Yes @ImTheKai ,
This change required to all other PostgreSQL branches.
Yes @nastena1606,
The Linux watchdog is mandatory for all production systems. Applicable for both Ubuntu/Debian and Redhat clones. It is already part of the pull request
fdf7425#diff-df4a6b913be84529e801fcdb19917e2a4c9e46886865a001e51c51b30aa8c72b

Copy link
Contributor

@nastena1606 nastena1606 left a comment

Choose a reason for hiding this comment

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

LGTM

…allation accidently
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.

None yet

4 participants