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

Increase Rate-limit config #5460

Merged
merged 1 commit into from
Aug 31, 2023
Merged

Conversation

GraysonWu
Copy link
Contributor

Fixes #5446

100 packets per second and 200 burst are not enough.
Increase to 500 packets per second and 1000 burst.

100 packets per second and 200 burst are not enought.
Increase to 500 packets per second and 1000 burst.

Signed-off-by: graysonwu <wgrayson@vmware.com>
@GraysonWu GraysonWu requested a review from tnqn August 31, 2023 03:25
Copy link
Member

@tnqn tnqn left a comment

Choose a reason for hiding this comment

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

LGTM

I tested with logging enabled and 2100+ sessions per second, the CPU usage of related processes are:
76% ovs-vswitchd
36% antrea-agent

Logging is working correctly, with rate-limiting and burst applied:

# awk -F "." '{print $1}' np.log |uniq -c
   1403 2023/08/31 06:48:26
    500 2023/08/31 06:48:27
    500 2023/08/31 06:48:28
    500 2023/08/31 06:48:29
    500 2023/08/31 06:48:30
    500 2023/08/31 06:48:31
    500 2023/08/31 06:48:32
    500 2023/08/31 06:48:33
    500 2023/08/31 06:48:34
    500 2023/08/31 06:48:35
    500 2023/08/31 06:48:36

@tnqn tnqn added action/backport Indicates a PR that requires backports. action/release-note Indicates a PR that should be included in release notes. labels Aug 31, 2023
@tnqn
Copy link
Member

tnqn commented Aug 31, 2023

/test-all
/test-windows-all
/test-ipv6-all

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action/backport Indicates a PR that requires backports. action/release-note Indicates a PR that should be included in release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pod's DNS resolution becomes slow after applying FQDN rules to it
2 participants