Skip to content
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

flaky: e2e TestEGUpgrade/EnvoyShutdown #3262

Closed
shawnh2 opened this issue Apr 24, 2024 · 4 comments · Fixed by #3646
Closed

flaky: e2e TestEGUpgrade/EnvoyShutdown #3262

shawnh2 opened this issue Apr 24, 2024 · 4 comments · Fixed by #3646
Assignees
Milestone

Comments

@shawnh2
Copy link
Contributor

shawnh2 commented Apr 24, 2024

https://github.com/envoyproxy/gateway/actions/runs/8811709532/job/24187972908?pr=2665#step:6:17406

@shawnh2 shawnh2 added triage help wanted Extra attention is needed area/testing and removed triage labels Apr 24, 2024
@zirain zirain removed the help wanted Extra attention is needed label Apr 28, 2024
@arkodg arkodg modified the milestones: v1.1.0-rc1, v1.1.0 May 23, 2024
@shawnh2
Copy link
Contributor Author

shawnh2 commented Jun 25, 2024

@shawnh2 shawnh2 reopened this Jun 25, 2024
@arkodg
Copy link
Contributor

arkodg commented Jun 25, 2024

can we bump initDelaySeconds to 10 ?

@guydc
Copy link
Contributor

guydc commented Jul 9, 2024

Sorry for the late reply, I missed that CC.

The issue this time appears to be connection reset/refused and not 404s:

{"ts":1719283813.373276,"level":"err","r":177,"file":"http_client.go","line":1098,"msg":"Read error","err":"read tcp 172.18.0.1:35884->172.18.255.208:80: read: connection reset by peer","size":0,"header_len":0,"dest":{"IP":"172.18.255.208","Port":80,"Zone":""},"url":"http://172.18.255.208/envoy-shutdown","thread":11,"run":0}
{"ts":1719283813.373384,"level":"err","r":313,"file":"http_client.go","line":1098,"msg":"Read error","err":"read tcp 172.18.0.1:35904->172.18.255.208:80: read: connection reset by peer","size":0,"header_len":0,"dest":{"IP":"172.18.255.208","Port":80,"Zone":""},"url":"http://172.18.255.208/envoy-shutdown","thread":35,"run":0}
{"ts":1719283813.373564,"level":"err","r":174,"file":"http_client.go","line":1098,"msg":"Read error","err":"read tcp 172.18.0.1:35872->172.18.255.208:80: read: connection reset by peer","size":0,"header_len":0,"dest":{"IP":"172.18.255.208","Port":80,"Zone":""},"url":"http://172.18.255.208/envoy-shutdown","thread":8,"run":0}
{"ts":1719283813.374018,"level":"err","r":177,"file":"http_client.go","line":951,"msg":"Unable to connect","dest":{"IP":"172.18.255.208","Port":80,"Zone":""},"err":"dial tcp 172.18.255.208:80: connect: connection refused","numfd":58,"thread":11,"run":0}

I'm not sure that increasing the readiness time will help in this case. This could be an issue with metallb/kube-proxy forwarding requests to terminated pods.

@shawnh2
Copy link
Contributor Author

shawnh2 commented Jul 17, 2024

closed in favour of #3764

@shawnh2 shawnh2 closed this as completed Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants