Skip to content

Clarify RateLimit Source IP Client Selector docs#1145

Merged
arkodg merged 1 commit intoenvoyproxy:mainfrom
arkodg:rl-ip-spec-clarify
Mar 14, 2023
Merged

Clarify RateLimit Source IP Client Selector docs#1145
arkodg merged 1 commit intoenvoyproxy:mainfrom
arkodg:rl-ip-spec-clarify

Conversation

@arkodg
Copy link
Copy Markdown
Contributor

@arkodg arkodg commented Mar 14, 2023

No description provided.

Signed-off-by: Arko Dasgupta <arko@tetrate.io>
@arkodg arkodg requested a review from a team as a code owner March 14, 2023 03:50
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #1145 (39c2b16) into main (27d14f8) will increase coverage by 0.12%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main    #1145      +/-   ##
==========================================
+ Coverage   62.55%   62.68%   +0.12%     
==========================================
  Files          72       72              
  Lines        9596     9596              
==========================================
+ Hits         6003     6015      +12     
+ Misses       3179     3168      -11     
+ Partials      414      413       -1     

see 2 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Copy Markdown
Member

@zirain zirain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@arkodg arkodg merged commit 2961990 into envoyproxy:main Mar 14, 2023
@arkodg arkodg deleted the rl-ip-spec-clarify branch March 14, 2023 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants