Skip to content

fix: limit concurrent half-open probes - #1281

Merged
RUKAYAT-CODER merged 3 commits into
rinafcode:mainfrom
poaspergillus:fix-1166-half-open-probes
Aug 30, 2026
Merged

fix: limit concurrent half-open probes#1281
RUKAYAT-CODER merged 3 commits into
rinafcode:mainfrom
poaspergillus:fix-1166-half-open-probes

Conversation

@poaspergillus

@poaspergillus poaspergillus commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add a dedicated configurable limit for concurrent HALF_OPEN circuit-breaker probes.
  • Fail fast with the existing fallback behavior when the probe limit is reached.
  • Protect lazy ethers loading with a single-probe circuit breaker and clear failed module loads so recovery can retry.
  • Add focused coverage for HALF_OPEN probe limiting and ethers-service integration.

Validation

Closes #1166

@drips-wave

drips-wave Bot commented Aug 29, 2026

Copy link
Copy Markdown

@poaspergillus Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@RUKAYAT-CODER

Copy link
Copy Markdown
Contributor

Well done on the job done so far!
Kindly fix workflow to pass

@poaspergillus

Copy link
Copy Markdown
Contributor Author

@RUKAYAT-CODER

Thank you. I’ve fixed the workflow issue, and all 6 PR checks for #1281 are now passing.

I’ve also applied to the other open issues in this project, including the ones I submitted today.

Since I’m already familiar with the codebase and workflow, I’d appreciate being considered for those as well.

I’ll continue submitting focused changes, proper tests, and clean PRs.

@RUKAYAT-CODER

Copy link
Copy Markdown
Contributor

Thank you for contributing to the project.

@RUKAYAT-CODER
RUKAYAT-CODER merged commit 673ca84 into rinafcode:main Aug 30, 2026
6 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.

Limit concurrent half-open probes in circuit breaker

2 participants