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

Support for Distributed System #675

Closed
mayankagarwals opened this issue Sep 2, 2022 · 7 comments
Closed

Support for Distributed System #675

mayankagarwals opened this issue Sep 2, 2022 · 7 comments

Comments

@mayankagarwals
Copy link

Hey,

This is a request to understand the current distributed systems support from opossum.
We can store the latest rolling window state in a common store, but does it make sense for opossum to provide it out of the box?

@lance lance added the question label Sep 2, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2022

This issue is stale because it has been open 30 days with no activity.

@gopiprasanthpotipireddy
Copy link

gopiprasanthpotipireddy commented Oct 23, 2022

@mayankagarwals I observed almost every top used circuit breaker module in other eco system also are reluctant to add this feature.

**Discussion Thread:

.NET Polly**
App-vNext/Polly#287 (comment)

Java : resilience4j
resilience4j/resilience4j#419

This Python package has optional redis cache support for distributed circuit breaker. I think the above modules are avoiding this for a reason but it is not unavoidable to deploy services in cluster these days. we can opt for the proxy/load balancer config over app config like istio in kubernetes

https://github.com/danielfm/pybreaker

@lance
Copy link
Member

lance commented Oct 24, 2022

@gopiprasanthpotipireddy @mayankagarwals we do have the ability to store and reload a circuit's state. It is discussed in the README.md here https://github.com/nodeshift/opossum#breaker-state-initialization. We do not want to pull external dependencies into this module, so the integration with redis or other state stores is left up to the user.

@github-actions
Copy link
Contributor

This issue is stale because it has been open 30 days with no activity.

@jdmarshall
Copy link
Contributor

I think at some point we will discontinue circuit breakers in favor of putting rate limiting in front of more services, instead of just entrypoints. Having two classes of service for rate control gets very weird especially when services are called both by external and internal processes.

@github-actions
Copy link
Contributor

This issue is stale because it has been open 30 days with no activity.

@lance
Copy link
Member

lance commented Jun 29, 2023

@lholmquist I am going to close this as wontfix as there seem to be no plans to address it. Please feel free to reopen it if you think otherwise.

@lance lance added the wontfix label Jun 29, 2023
@lance lance closed this as completed Jun 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants