-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Scanning port 10247 lead to tcp connection 502 error #9815
Conversation
This issue is currently awaiting triage. If Ingress contributors determines this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Welcome @lijiecm! |
Hi @lijiecm. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Can you clarify what you mean by scanning a port and how it relates to a 502 when backends are not there? |
for example ,you have a process used tcp-services, when you update your pod ,ingress controller will be update your tcp backends, but if you requests ingress http://127.0.0.1:10247, lua will use strings "GET / HTTP/1.1" to clean your tcp backends , this time if you have a http service update ,ingress nginx will be reload, all of your tcp service will be 502 error. |
makes sense. @lijiecm can you sign the CLA please? Thanks |
Done Thanks |
/ok-to-test |
/lgtm |
@lijiecm there is a fix on lua code, can you take a look please? Thanks!! |
@rikatz done, It looks like the checks passed |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lijiecm, rikatz 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:
Approvers can indicate their approval by writing |
/cherry-pick release-1.8 |
@strongjz: new pull request created: #10222 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
If you have a safe process to scan local port 10247, Ingress nginx will clean up your backend endpoints. At this time, While you change your http project, Ingress nginx will be reloaded and your tcp connection will have a 502 error.