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

Proposal: Envoy Admission Control calculation to use latency statistics #36415

Open
marcusoyang opened this issue Oct 2, 2024 · 4 comments
Open
Assignees
Labels
area/admission_control enhancement Feature requests. Not bugs or questions.

Comments

@marcusoyang
Copy link

tl:dr

Add latency statistics into the Admission Control filter success rate calculation

Context

To make the Admission Control capability more effective in the context of slow dependency response times, the proposal is to add to the Admission Control filter envoy/source/extensions/filters/http/admission_control at main · envoyproxy/envoy and add response latency statistics in its successRateThreshold calculation.

Currently, success rate is derived only from the HTTP response status codes of a dependency. This heuristic is effective when response latency is relatively fast, however in the situation when a dependency’s response latency is degraded, this would be useful.

@marcusoyang marcusoyang added enhancement Feature requests. Not bugs or questions. triage Issue requires triage labels Oct 2, 2024
@kyessenov kyessenov added area/admission_control and removed triage Issue requires triage labels Oct 3, 2024
@kyessenov
Copy link
Contributor

CC @tonya11en

@tonya11en
Copy link
Member

@marcusoyang I'm happy to explore this with you. Have you taken a look at Adaptive Concurrency
Control
? That mechanism uses sampled latencies to make adjustments to the concurrency limit (essentially circuit breaking) thresholds.

I'm curious whether that meets your requirements and if not, why.

@tonya11en
Copy link
Member

/assign @tonya11en

@tonya11en
Copy link
Member

It may also help to get more context around what "slow dependency response times" means.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/admission_control enhancement Feature requests. Not bugs or questions.
Projects
None yet
Development

No branches or pull requests

3 participants