Global RateLimit Xds translation#726
Conversation
b8ff9c2 to
f7288b8
Compare
Codecov Report
@@ Coverage Diff @@
## main #726 +/- ##
==========================================
+ Coverage 63.93% 63.97% +0.03%
==========================================
Files 51 52 +1
Lines 6863 7136 +273
==========================================
+ Hits 4388 4565 +177
- Misses 2201 2286 +85
- Partials 274 285 +11
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
f7288b8 to
2af765e
Compare
internal/xds/translator/testdata/out/xds-ir/ratelimit.clusters.yaml
Outdated
Show resolved
Hide resolved
internal/ir/xds.go
Outdated
There was a problem hiding this comment.
@arkodg I don't understand why RateLimit doesn't use any of the RateLimitFilter fields. For example, #813 uses JwtAuthenticationFilterProvider instead of recreating the type in the IR. Since we own extension APIs, e.g. RateLimitFilter, we should leverage these types as much as possible.
There was a problem hiding this comment.
creating a decoupling from Front End as well as Back End API, that was the main goal of the IR, which is why we created custom structures even for known structures such as StringMatch
There was a problem hiding this comment.
as you mentioned, since the project owns the structure, should be safe to reuse it, but the structure fields are also arranged differently to facilitate better translation (more machine friendly than user friendly)
|
@arkodg regarding #726 (comment), thanks for the suggestion. #813 focuses solely on the xds IR changes. I plan to submit a follow-on PR that adds AuthenticationFilter support to the xds translator. I'll look at following the pattern here for the follow-on PR. |
* Enhance `XdsIR` with `RateLimit` to hold rate limiting config. * Translate IR field into route level rate limit actions * Add `BuildRateLimitServiceConfig` which translates the XdsIR into configuration for the envoy rate limit service. Relates to envoyproxy#670 Signed-off-by: Arko Dasgupta <arko@tetrate.io>
Signed-off-by: Arko Dasgupta <arko@tetrate.io>
Signed-off-by: Arko Dasgupta <arko@tetrate.io>
Signed-off-by: Arko Dasgupta <arko@tetrate.io>
Signed-off-by: Arko Dasgupta <arko@tetrate.io>
Signed-off-by: Arko Dasgupta <arko@tetrate.io>
Signed-off-by: Arko Dasgupta <arko@tetrate.io>
Signed-off-by: Arko Dasgupta <arko@tetrate.io>
Enhance
XdsIRwithRateLimitto hold rate limiting config.Translate IR field into route level rate limit actions
This PR specifically implements this points
stated in #858
Relates to #670
Signed-off-by: Arko Dasgupta arko@tetrate.io