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 the aggression parameter within the LoadBalancerSettings configuration. #51093

Open
xocodergit opened this issue May 16, 2024 · 0 comments

Comments

@xocodergit
Copy link

xocodergit commented May 16, 2024

(This is used to request new product features, please visit https://github.com/istio/istio/discussions for questions on using Istio)

Describe the feature request
Istio now supports the envoy slow_start_window parameters through the LoadBalancerSettings.warmupDurationSecs feature. In most cases, this configuration is sufficient for effective service warm-up. However, because of the lack of aggression parameter, there are certain scenarios where this mechanism may not yield the ideal result. For instance, applications with built-in caching mechanisms, where cache building is request-triggered, might experience a non-linear decrease in response latency as the cache becomes populated. In such cases, the initial performance drop due to cache build-up can be significant before it stabilizes to optimal response times. To achieve overall better performance, it is preferable to assign fewer requests during the initial phase of the application startup, then rapidly increase the volume of requests to regular levels once the cache has been built. This is precisely what the aggression parameter can do.

Describe alternatives you've considered
I've noticed the issue reported at #41039. While an EnvoyFilter patch can be used to adjust the aggression parameter, a more elegant solution would be to allow configuration directly within the DestinationRule.

Affected product area (please put an X in all that apply)

[ ] Ambient
[ ] Docs
[ ] Dual Stack
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[x] User Experience
[ ] Developer Infrastructure

Affected features (please put an X in all that apply)

[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane

Additional context

I'd like raise a PR to add support for "aggression" parameter :)

@xocodergit xocodergit changed the title Aggression parameter supporting of warmup setting Support for the aggression parameter within the warmup setting configuration. May 16, 2024
@xocodergit xocodergit changed the title Support for the aggression parameter within the warmup setting configuration. Support for the aggression parameter within the LoadBalancerSettings configuration. May 16, 2024
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

2 participants