Open weli-l opened 4 weeks ago
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please assign hzxuzhonghu for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.
Attention: Patch coverage is 0%
with 180 lines
in your changes missing coverage. Please review.
Flag | Coverage Δ | |
---|---|---|
unittests | 36.02% <0.00%> (+4.01%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Files | Coverage Δ | |
---|---|---|
pkg/controller/bypass/bypass_controller.go | 0.00% <0.00%> (ø) |
|
pkg/controller/controller.go | 0.00% <0.00%> (ø) |
|
pkg/controller/netns/netns.go | 0.00% <0.00%> (ø) |
|
pkg/controller/manage/kmesh_manage.go | 0.00% <0.00%> (ø) |
What type of PR is this?
What this PR does / why we need it: support Kmesh manage in pod level Which issue(s) this PR fixes: Fixes #325
Special notes for your reviewer:
How to enable kmesh management at pod level
How to judge whether pod is managed by Kmesh
Due to the feature of the bypass, the impact of the bypass needs to be considered comprehensively
Label priority(From high to low)
Annotation priority(From high to low,and it is only be used for user to judge whether pod is managed by kmesh)
Does this PR introduce a user-facing change?: