Skip to content

[*] fix Reap() method to update Recovery cache after processing the previous value#815

Merged
pashagolub merged 1 commit intocybertec-postgresql:masterfrom
0xgouda:fix-last-recovery-check
Jun 24, 2025
Merged

[*] fix Reap() method to update Recovery cache after processing the previous value#815
pashagolub merged 1 commit intocybertec-postgresql:masterfrom
0xgouda:fix-last-recovery-check

Conversation

@0xgouda
Copy link
Copy Markdown
Collaborator

@0xgouda 0xgouda commented Jun 23, 2025

move the hostLastKnownStatusInRecovery[monitoredSource.Name] = monitoredSource.IsInRecovery update to after the processing of the last cached value
closes: #814

@0xgouda 0xgouda changed the title [*] fix Reap() method to update hostLastKnownStatuseInRecovery only after processing the previous value [*] fix Reap() method to update Recovery cache after processing the previous value Jun 23, 2025
@coveralls
Copy link
Copy Markdown

coveralls commented Jun 23, 2025

Pull Request Test Coverage Report for Build 15848529645

Details

  • 0 of 2 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.03%) to 61.329%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/reaper/reaper.go 0 2 0.0%
Totals Coverage Status
Change from base Build 15848273911: 0.03%
Covered Lines: 2742
Relevant Lines: 4471

💛 - Coveralls

@pashagolub pashagolub force-pushed the fix-last-recovery-check branch from 7023102 to 8b1cbf4 Compare June 24, 2025 10:55
@pashagolub pashagolub self-assigned this Jun 24, 2025
@pashagolub pashagolub added the bug Something isn't working label Jun 24, 2025
@pashagolub pashagolub merged commit 86d95fc into cybertec-postgresql:master Jun 24, 2025
5 checks passed
@0xgouda 0xgouda deleted the fix-last-recovery-check branch August 1, 2025 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error switching standby and primary metrics

3 participants