code

I think we still need a way to limit the overhead and memory of the token buckets. It's unacceptable to let it increases unlimited.

Thanks for this contribution. Dynamic descriptor support is a very complex problem in the local rate limit, considering various limitations. I have take a pass to current implementation, but before...

Thanks so much for this update. I think this make sense. Here are some high level suggestions to this (I think we are in the correct way, thanks): 1. Please...

Thanks for this great contribution. I am so happy to see this happens. I think we are at correct way after a quick check. But I prefer to review and...

> yeah I'm not sure but I could imagine checkActiveStaticCluster being too narrow of a check https://github.com/envoyproxy/envoy/blob/main/source/extensions/access_loggers/fluentd/config.cc#L42 cc @ohadvano @wbpcode Yeah, I think the checkActiveStaticCluster is too strict for the...

> Thank you for the clarification! > > If configuring the cluster in the bootstrap is indeed a requirement for this feature, it would be great to mention this explicitly...

Seems this proposal contains lots of features. What will be the initial target or is there a roadmap? Now, I am instersted in the 1 and 5. I think 5...

I can help with review of the API and code. It's happy to see this could be core part of Envoy. cc @botengyao cc @johnlanni

> To that end, I wonder if the https://github.com/envoyproxy org be willing to host a repository for an experimental MCP extension? If so, it would help to concentrate the energy...

> > To that end, I wonder if the https://github.com/envoyproxy org be willing to host a repository for an experimental MCP extension? If so, it would help to concentrate the...