-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 observe backend health status #225
Labels
enhancement
New feature or request
Comments
vicanso
added a commit
to vicanso/pingora
that referenced
this issue
Jul 13, 2024
andrewhavck
pushed a commit
that referenced
this issue
Aug 9, 2024
--- test: add test for upstream health observe --- renamed the function and added doc to make it intelligible --- fix clippy error --- Merge branch 'main' into main --- test: fix test for backend do update Co-authored-by: Tree Xie <tree.xie@outlook.com> Includes-commit: 1421c26 Includes-commit: 695d549 Includes-commit: 6a09b52 Includes-commit: 72d6ee0 Includes-commit: e6c2af0 Includes-commit: fb62869 Replicated-from: #325
andrewhavck
pushed a commit
that referenced
this issue
Aug 9, 2024
--- test: add test for upstream health observe --- renamed the function and added doc to make it intelligible --- fix clippy error --- Merge branch 'main' into main --- test: fix test for backend do update Co-authored-by: Tree Xie <tree.xie@outlook.com> Includes-commit: 1421c26 Includes-commit: 695d549 Includes-commit: 6a09b52 Includes-commit: 72d6ee0 Includes-commit: e6c2af0 Includes-commit: fb62869 Replicated-from: #325
escoffier
pushed a commit
to escoffier/pingora
that referenced
this issue
Sep 6, 2024
--- test: add test for upstream health observe --- renamed the function and added doc to make it intelligible --- fix clippy error --- Merge branch 'main' into main --- test: fix test for backend do update Co-authored-by: Tree Xie <tree.xie@outlook.com> Includes-commit: 1421c26 Includes-commit: 695d549 Includes-commit: 6a09b52 Includes-commit: 72d6ee0 Includes-commit: e6c2af0 Includes-commit: fb62869 Replicated-from: cloudflare#325
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What is the problem your feature solves, or the need it fulfills?
I want to get the status change of backend, and send a notification. Now it only prints a warn log. https://github.com/cloudflare/pingora/blob/main/pingora-load-balancing/src/lib.rs#L229, I can get status change from logger.
Is it possible to support callback function to observe it?
Describe the solution you'd like
Supports to set callbacks like
TlsAcceptCallbacks
.Describe alternatives you've considered
What other solutions, features, or workarounds have you considered that might also solve the issue?
What are the tradeoffs for these alternatives compared to what you're proposing?
Additional context
This could include references to documentation or papers, prior art, screenshots, or benchmark
results.
The text was updated successfully, but these errors were encountered: