Closed kmrmt closed 3 months ago
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
[CHATOPS:HELP] ChatOps commands.
/approve
- approve/format
- format codes and add licenses/gen-test
- generate test codes/label
- add labels/label actions/e2e-deploy
- run E2E deploy & integration testAttention: Patch coverage is 0%
with 1 lines
in your changes are missing coverage. Please review.
Project coverage is 18.90%. Comparing base (
6d4b505
) to head (99b3333
). Report is 9 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
pkg/gateway/lb/handler/grpc/handler.go | 0.00% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
[FOSSA] The scan result will be available at https://app.fossa.com/projects/custom%2B21465%2Fvald/refs/branch/fix%2Fgateway%2Flarge-topK-ratio-handling/abf8e6cd163a2432a7b1373a21e6dbbef04e394d
type | vald-agent-ngt | vald-lb-gateway | vald-discoverer | vald-manager-index |
---|---|---|---|---|
cpu | ![]() | ![]() | ![]() | ![]() |
heap | ![]() | ![]() | ![]() | ![]() |
Description
Update ratio handling with large top-K. Merge the logic ratio is 0 and ratio is less than 0.
Related Issue
Versions
Checklist
Special notes for your reviewer
Summary by CodeRabbit