Add RateLimitFilter support in k8s provider#908
Conversation
Codecov Report
@@ Coverage Diff @@
## main #908 +/- ##
==========================================
- Coverage 63.40% 63.26% -0.15%
==========================================
Files 53 53
Lines 7487 7556 +69
==========================================
+ Hits 4747 4780 +33
- Misses 2440 2468 +28
- Partials 300 308 +8
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
| expected: true, | ||
| }, | ||
| { | ||
| name: "httproute with one rateLimitfilter", |
There was a problem hiding this comment.
Can you also create a test case that has 1 AuthenFilter and 1 RateLimitFilter?
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>
|
@arkodg CI is failing b/c the controller-runtime cache is unable to sync. The cache is unable to sync b/c the RateLimitFilter CRD is not installed by the Creating an HTTPRoute that references a RateLimitFilter results in: You need to update |
thanks for helping debug this issue ! will make the change in the PR |
Signed-off-by: Arko Dasgupta <arko@tetrate.io>
Relates to #670
Signed-off-by: Arko Dasgupta arko@tetrate.io