Commit 2e089ff
authored
Fix qos not work (#6231)
* fix(qos): fix NAT Gateway QoS not working
* fix eip deletion not trigger qos again
This PR fixes multiple issues with NAT Gateway QoS functionality:
1. Fix matchall filter handle extraction regex
- matchall filter uses 'handle 0xNNNN' format, not 'fh' like u32 filters
- Changed regex from 'fh [0-9a-f:]+' to 'handle 0x[0-9a-fA-F]+'
2. Fix matchall filter line extraction when u32 filters exist
- When both u32 and matchall filters exist, 'grep -B2 matchall' captured wrong lines
- Use 'grep matchall.*flowid' to directly match the correct line
3. Use IFB + HTB instead of police for ingress QoS
- police drops packets exceeding rate limit (unfriendly to TCP)
- HTB queues packets and applies backpressure, allowing TCP to adapt smoothly
- This results in actual throughput close to the configured rate limit
4. Support decimal values for rateMax/burstMax (e.g., 1.5 Mbit/s)
- Convert burst from MB to bytes to avoid tc parsing issues
5. Add detailed parameter documentation for all QoS functions
- qos_add: 9 parameters
- qos_del: 7 parameters
- eip_*_qos_add: 4 parameters
- eip_*_qos_del: 1 parameter
6. Add DEBUG switch for QoS troubleshooting
---------
Signed-off-by: zbb88888 <jmdxjsjgcxy@gmail.com>1 parent fb80618 commit 2e089ff
File tree
10 files changed
+2481
-343
lines changed- charts
- kube-ovn-v2/crds
- kube-ovn/templates
- dist/images
- vpcnatgateway
- makefiles
- pkg/controller
- test/e2e/iptables-eip-qos
10 files changed
+2481
-343
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3583 | 3583 | | |
3584 | 3584 | | |
3585 | 3585 | | |
3586 | | - | |
| 3586 | + | |
| 3587 | + | |
3587 | 3588 | | |
3588 | 3589 | | |
3589 | | - | |
| 3590 | + | |
| 3591 | + | |
3590 | 3592 | | |
3591 | 3593 | | |
3592 | 3594 | | |
| |||
3646 | 3648 | | |
3647 | 3649 | | |
3648 | 3650 | | |
3649 | | - | |
| 3651 | + | |
| 3652 | + | |
3650 | 3653 | | |
3651 | 3654 | | |
3652 | | - | |
| 3655 | + | |
| 3656 | + | |
3653 | 3657 | | |
3654 | 3658 | | |
3655 | 3659 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3627 | 3627 | | |
3628 | 3628 | | |
3629 | 3629 | | |
3630 | | - | |
| 3630 | + | |
| 3631 | + | |
3631 | 3632 | | |
3632 | 3633 | | |
3633 | | - | |
| 3634 | + | |
| 3635 | + | |
3634 | 3636 | | |
3635 | 3637 | | |
3636 | 3638 | | |
| |||
3690 | 3692 | | |
3691 | 3693 | | |
3692 | 3694 | | |
3693 | | - | |
| 3695 | + | |
| 3696 | + | |
3694 | 3697 | | |
3695 | 3698 | | |
3696 | | - | |
| 3699 | + | |
| 3700 | + | |
3697 | 3701 | | |
3698 | 3702 | | |
3699 | 3703 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3877 | 3877 | | |
3878 | 3878 | | |
3879 | 3879 | | |
3880 | | - | |
| 3880 | + | |
| 3881 | + | |
3881 | 3882 | | |
3882 | 3883 | | |
3883 | | - | |
| 3884 | + | |
| 3885 | + | |
3884 | 3886 | | |
3885 | 3887 | | |
3886 | 3888 | | |
| |||
3940 | 3942 | | |
3941 | 3943 | | |
3942 | 3944 | | |
3943 | | - | |
| 3945 | + | |
| 3946 | + | |
3944 | 3947 | | |
3945 | 3948 | | |
3946 | | - | |
| 3949 | + | |
| 3950 | + | |
3947 | 3951 | | |
3948 | 3952 | | |
3949 | 3953 | | |
| |||
0 commit comments