-
Notifications
You must be signed in to change notification settings - Fork 39.8k
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
fix IPVS low throughput issue #71114
Conversation
Here are the ab test results: Server Software: nginx/1.15.5
Server Hostname: 10.0.0.147
Server Port: 8080
Document Path: /
Document Length: 612 bytes
Concurrency Level: 300
Time taken for tests: 120.017 seconds
Complete requests: 48983
Failed requests: 0
Total transferred: 41390635 bytes
HTML transferred: 29977596 bytes
Requests per second: 408.13 [#/sec] (mean)
Time per request: 735.051 [ms] (mean)
Time per request: 2.450 [ms] (mean, across all concurrent requests)
Transfer rate: 336.79 [Kbytes/sec] received
Connection Times (ms)
min mean[+/-sd] median max
Connect: 0 722 457.9 999 3009
Processing: 0 9 23.3 1 424
Waiting: 0 9 23.2 1 424
Total: 0 732 449.6 1000 3201
Percentage of the requests served within a certain time (ms)
50% 1000
66% 1000
75% 1000
80% 1001
90% 1002
95% 1004
98% 1007
99% 1198
100% 3201 (longest request) After: Server Software: nginx/1.15.5
Server Hostname: 10.0.0.147
Server Port: 8080
Document Path: /
Document Length: 612 bytes
Concurrency Level: 300
Time taken for tests: 106.312 seconds
Complete requests: 580000
Failed requests: 0
Total transferred: 490100000 bytes
HTML transferred: 354960000 bytes
Requests per second: 5455.62 [#/sec] (mean)
Time per request: 54.989 [ms] (mean)
Time per request: 0.183 [ms] (mean, across all concurrent requests)
Transfer rate: 4501.95 [Kbytes/sec] received
Connection Times (ms)
min mean[+/-sd] median max
Connect: 0 28 392.7 0 63157
Processing: 5 25 88.6 23 25652
Waiting: 4 25 88.6 23 25652
Total: 15 54 409.7 24 63348
Percentage of the requests served within a certain time (ms)
50% 24
66% 25
75% 26
80% 27
90% 30
95% 38
98% 56
99% 1025
100% 63348 (longest request) |
/cc @m1093782566 @jsravn |
@Lion-Wei: GitHub didn't allow me to request PR reviews from the following users: jsravn. Note that only kubernetes members and repo collaborators can review this PR, and authors cannot review their own PRs. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Good improvement, THANKS! /lgtm /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Lion-Wei, m1093782566 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest Review the full test history for this PR. Silence the bot with an |
/milestone v1.13 |
/priority important-soon |
Hello everyone:
Thank you. |
Following-up on @yyx's comment above for posterity. The patch mentioned above didn't make it to the kernel but there are two recently merged patches worth highlighting. One of them fixes the 1 second delay issue and the other fixes dropped packets when stale connection entries in the IPVS table are used: |
What type of PR is this?
/kind bug
What this PR does / why we need it:
This pr make IPVS proxier set net/ipv4/vs/conn_reuse_mode to 0 by default, which will fix the IPVS low throughput issue.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #70747
Special notes for your reviewer:
Does this PR introduce a user-facing change?: