Closed windnod closed 2 months ago
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
No data about Duplication
Attention: Patch coverage is 40.00000%
with 3 lines
in your changes are missing coverage. Please review.
Project coverage is 52.41%. Comparing base (
c8778fc
) to head (6938de3
). Report is 55 commits behind head on master.
Files | Patch % | Lines |
---|---|---|
pkg/yurthub/proxy/remote/loadbalancer.go | 0.00% | 2 Missing :warning: |
pkg/yurthub/proxy/local/local.go | 0.00% | 0 Missing and 1 partial :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@windnod Thanks for posting pull request. I have commented the issue #1963 , and we need to have more discussion about this case before we merge this pull request. sorry to response you lately.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
What type of PR is this?
/kind bug
What this PR does / why we need it:
skip the check for the same path list requests, when the localproxy use the canCacheFor() method
Which issue(s) this PR fixes:
Fixes #1963