go-control-plane
go-control-plane copied to clipboard
Include a sample Rate-Limit config xDS server
I am working on https://github.com/envoyproxy/ratelimit/issues/201 which is a feature for updating Rate Limit Service configurations dynamically through an xDS server.
As per https://github.com/envoyproxy/ratelimit/issues/201 I thought of adding the server-related implementation within the go-control-plane repository. Then we could re-use most of the readily available functions here. Hence, I created this git issue to discuss how we can achieve this.
Please share your thoughts on this.
Hi @renuka-fernando this would be awesome!
I'd actually like to have the rate limit service become a first class citizen of this repo since it's needed for control-plane functionality :)
This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or "no stalebot" or other activity occurs. Thank you for your contributions.
hi @alecholmez,
Is it possible to add the "no stalebot" label?
This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or "no stalebot" or other activity occurs. Thank you for your contributions.
hi @renuka-fernando can you kindly add this to java-control-plane too, Please.. It will be super helpful.
hi @renuka-fernando can you kindly add this to java-control-plane too, Please.. It will be super helpful.
sure, I will. Once @alecholmez confirms the changes on go control plane, I will have a look at the java control plane as well.
kindly, have this included in java control plane too, i am eagerly awating to try it out.
Hi @alecholmez,
Missed one more thing, we have to have a way to generate pb files. As we discussed here https://github.com/envoyproxy/go-control-plane/pull/598#discussion_r1008130688
So could you reopen this again to track it? Source proto files: https://github.com/envoyproxy/ratelimit/pull/371
@renuka-fernando okay great! Thanks for following up with that.