Ratelimit bug fix. - Githubissues

kmesh-net / kmesh

High Performance ServiceMesh Data Plane Based on Programmable Kernel

https://kmesh.net

Apache License 2.0

466 stars 70 forks source link

Ratelimit bug fix. #1018

Closed yuanqijing closed 1 week ago

yuanqijing commented 2 weeks ago

What type of PR is this? fix ratelimit compile failed on oe.

What this PR does / why we need it:

Which issue(s) this PR fixes: Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 48.01%. Comparing base (efb1588) to head (c24ac00). Report is 162 commits behind head on main.

see 23 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 55d4e65...c24ac00. Read the comment docs.

yuanqijing commented 2 weeks ago

/unhold

tacslon commented 1 week ago

Need some testing on enhanced kernel, otherwise LGTM.

yuanqijing commented 1 week ago

@tacslon hi i have test on openEuler 2303: before:

after:

kmesh-bot commented 1 week ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hzxuzhonghu

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: - ~~[OWNERS](https://github.com/kmesh-net/kmesh/blob/main/OWNERS)~~ [hzxuzhonghu] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment