-
Notifications
You must be signed in to change notification settings - Fork 370
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow Node probe all local Pods #175
Conversation
Thanks for your PR. The following commands are available:
|
/test-e2e |
Maybe revise the commit message to replace table 90 and 110 with the readable table names |
/test-e2e |
Consider "Allow Node probe all local Pods" as the title |
Sure, I will change. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@jianjuns please take a look if this matches your expectation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Install Openflow entry on ingress rule table to allow the liveness probe traffic that is sent from node to local Pods. The flow entry is installed on the ingress rule table, matching local gateway IP as the source address, and the action is to resubmit the packets to L2 forwarding output table. Signed-off-by: wenyingd <[email protected]> Co-Authored-By: Quan Tian <[email protected]>
/test-e2e |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Install Openflow entry on ingress rule table to allow the liveness probe traffic
that is sent from node to local Pods. The flow entry is installed on the ingress
rule table, matching local gateway IP as the source address, and the action is
to resubmit the packets to L2 forwarding output table.
Fixes #176