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

all the descriptors are consolidated in a single descriptor #62

Closed
eguzki opened this issue May 12, 2022 · 2 comments · Fixed by #396
Closed

all the descriptors are consolidated in a single descriptor #62

eguzki opened this issue May 12, 2022 · 2 comments · Fixed by #396

Comments

@eguzki
Copy link
Contributor

eguzki commented May 12, 2022

For the following Envoy's RLS request:

domain: example.org    
descriptors:           
  - entries:           
    - C: VALUE         
  - entries:           
    - A: VALUE         
    - B: VALUE         

the following ratelimit config would be activated and it should not.

conditions: []  
max_value: 1                      
seconds: 60                       
variables: ["A", "B", "C"]             
namespace: example.org            
@eguzki eguzki moved this to Needs refinement in Kuadrant Service Protection Jan 27, 2023
@alexsnaps alexsnaps moved this from Needs refinement to To do in Kuadrant Service Protection Apr 13, 2023
@alexsnaps
Copy link
Member

@didierofrivia may be that could be interesting for you to look into…

@alexsnaps
Copy link
Member

alexsnaps commented Nov 25, 2024

See #385 ... maybe we should consider having descriptors as a root binding and have it as a List of Map<String, String>... ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Status: To do
Development

Successfully merging a pull request may close this issue.

2 participants