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

Adding check_streaming_delta #152

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

infraweavers
Copy link

We run a 4 node setup where we have a hot standby and a cascading replication setup like:

A -> B
|
C -> D

where A is the master.

We want to monitor the delays at various points seperately, i.e. A->B should have a low delta, whereas A->C can be longer, whereas C->D should also be low delta.

We couldn't get our desired monitoring with any of the existing checks so we had to add one.

This lets a pginstance be monitored based on receiving and sending WAL, so we can monitor the delay between our cascading replication servers. We've also added the ability to limit based on being in the same network as the cluster that's being checked from which is useful for checking only local replicas if you have a hub-and-spoke-style replication strategy.

This lets a pginstance be monitored based on receiving and sending WAL, so we can monitor the delay between our cascading replication servers. We've also added the ability to limit based on being in the same network as the cluster that's being checked from which is useful for checking only local replicas if you have a hub-and-spoke-style  replication strategy
We need to call `add_ok` once for each instance of `$db` otherwise perfdata doesn't work at all, more frequently and it is duplicated
…positions, which causes this to go critical. In reality, this is not for checking pg_basebackup status
@infraweavers
Copy link
Author

@turnstep Curious as to whether there is any feedback on this PR? Thanks

@turnstep
Copy link
Contributor

Sorry, been to busy to look this over, will try this weekend.

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