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

Fix check_archive_ready for version 10 and above #204

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mbanck
Copy link
Contributor

@mbanck mbanck commented Oct 4, 2023

This addresses issue #150.

@mbanck mbanck force-pushed the check_archive_ready_v10 branch from 680aa4a to 264458c Compare October 4, 2023 16:12
The archive_ready check uses the wal_files check with dedicated options -
however, since wal_files was fixed for version 10 in 8e6b3c2 it called
pg_ls_waldir() even for the archive_ready check which is not
appropriate as this only lists files in pg_wal and not in
pg_wal/archive_status.

This change makes the wal_files check call pg_ls_archive_statusdir()
instead if $extrabit (i.e. .ready) is set, fixing issue bucardo#150.
@mbanck mbanck force-pushed the check_archive_ready_v10 branch from 264458c to 6d69b98 Compare October 4, 2023 16:19
@mbanck-ntap
Copy link

Ping, any updated on this?

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.

2 participants