Skip to content

Explain Postgres restart acknowledgment and verification - #971

Merged
sdairs merged 2 commits into
codex/focused-912from
codex/focused-932
Sep 17, 2026
Merged

sdairs merged 2 commits into
codex/focused-912from
codex/focused-932

Conversation

@sdairs

@sdairs sdairs commented Sep 15, 2026

Copy link
Copy Markdown
Collaborator

Document how to verify an accepted Postgres restart and configuration change. Re-read stored settings with config get, check active Postgres values with SHOW, and compare pg_postmaster_start_time() before/after a restart. Service state or readiness alone does not confirm a restart.

README and help use current ID/name selectors and link the official PostgreSQL sources. Runtime behavior, JSON and wait options are unchanged.

Validation: three command-tree structural tests, formatting and diff checks passed.

Closes #932

Appended to native GitHub stack #876.

Combined local validation

Combined stack head e20d63b3af4c8541a6f597d9494b530f2c56b666 passed all required local checks: formatting, CLI clippy/tests, telemetry-disabled check and all-target clippy, API/analyzer all-target clippy/tests, all-feature workspace build, Python classifier/drift tests, and diff checks.

Test totals: 1991 CLI, 672 API/analyzer, and 93 Python tests (2756 total). Ignored/live tests are excluded from these totals.

Snapshot API access was revalidated successfully on the exact heads of PRs #940 and #944 after the environment feature flag was enabled. Run links and coverage details are recorded in tracker #939.

Stack order, exact heads and explicit issue Development relationships were verified in the implementation tracker.

@sdairs
sdairs added this pull request to stack #876 September 15, 2026 16:21
@sdairs
sdairs marked this pull request as ready for review September 16, 2026 13:14
Carry existing lower-stack README corrections and ancestry forward. The merged tree matches the independently verified cumulative stack result.

@iskakaushik iskakaushik left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@sdairs
sdairs merged commit a65a9ca into main Sep 17, 2026
10 checks 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.

Explain Postgres restart acknowledgment and verification

2 participants