Skip to content

Conversation

@dragomirp
Copy link
Contributor

Partial port of #1049

Use the Patroni API to detect restart need

Checklist

  • I have added or updated any relevant documentation.
  • I have cleaned any remaining cloud resources from my accounts.

@dragomirp dragomirp added the enhancement New feature, UI change, or workload upgrade label Sep 18, 2025
@codecov
Copy link

codecov bot commented Sep 18, 2025

Codecov Report

❌ Patch coverage is 65.21739% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.70%. Comparing base (7542643) to head (aebd98d).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
src/charm.py 53.57% 12 Missing and 1 partial ⚠️
src/cluster.py 81.25% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1165      +/-   ##
==========================================
- Coverage   75.76%   75.70%   -0.06%     
==========================================
  Files          16       16              
  Lines        4163     4174      +11     
  Branches      631      628       -3     
==========================================
+ Hits         3154     3160       +6     
- Misses        786      793       +7     
+ Partials      223      221       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dragomirp
Copy link
Contributor Author

We should most probably update Patroni before we can port this.

@dragomirp dragomirp closed this Sep 23, 2025
@dragomirp dragomirp deleted the dpe-7726-restart-pending branch October 7, 2025 02:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature, UI change, or workload upgrade Libraries: OK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants