Skip to content

PG-1529 Fixed Patroni config file example and troubleshooting advice. - #1028

Merged
nastena1606 merged 4 commits into
17from
PG-1529-17
Aug 19, 2026
Merged

PG-1529 Fixed Patroni config file example and troubleshooting advice.#1028
nastena1606 merged 4 commits into
17from
PG-1529-17

Conversation

@nastena1606

Copy link
Copy Markdown
Contributor

This PR incorporates changes suggested in #779

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the Patroni HA documentation to provide a more accurate configuration example and troubleshooting guidance, aligned with the improvements discussed in #779, and makes a minor wording refinement in the installation overview.

Changes:

  • Updates the Patroni patroni.yml example to include pg_hba/recovery_conf under bootstrap.dcs.postgresql and switches auth examples from md5 to scram-sha-256.
  • Expands the explanatory text and troubleshooting guidance, including DCS update instructions via patronictl edit-config.
  • Refines wording in the “Tar download” installation section for clarity.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
docs/solutions/ha-patroni.md Adjusts Patroni config example and adds/updates explanatory + troubleshooting guidance (including auth method changes and DCS update workflow).
docs/installing.md Rewords the tarball-installation intro sentence for clarity.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/solutions/ha-patroni.md Outdated
Comment thread docs/solutions/ha-patroni.md Outdated

@jobinau jobinau left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good to publish

@jobinau jobinau left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approves the changes

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@nastena1606
nastena1606 temporarily deployed to PG-1529-17 - postgresql-docs-preview-17 PR #1028 August 19, 2026 17:39 — with Render Destroyed
@nastena1606
nastena1606 merged commit 056bf15 into 17 Aug 19, 2026
1 check passed
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.

3 participants