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

tmp: test unreleased GDR #6072

Closed
wants to merge 1 commit into from
Closed

tmp: test unreleased GDR #6072

wants to merge 1 commit into from

Conversation

rainest
Copy link
Contributor

@rainest rainest commented May 22, 2024

Run tests against proposed Kong/go-database-reconciler#88

@rainest rainest added the do not merge let the author merge this, don't merge for them. label May 22, 2024
@rainest rainest requested a review from a team as a code owner May 22, 2024 21:55
@rainest rainest added the ci/run-e2e Trigger e2e test run from PR label May 22, 2024
@team-k8s-bot
Copy link
Collaborator

E2E (targeted) tests with KIND-based clusters were started at https://github.com/Kong/kubernetes-ingress-controller/actions/runs/9198897002

@team-k8s-bot team-k8s-bot removed the ci/run-e2e Trigger e2e test run from PR label May 22, 2024
Copy link

codecov bot commented May 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (main@eb04ce9). Learn more about missing BASE report.

Current head 03399fb differs from pull request most recent head f545c84

Please upload reports for the commit f545c84 to get more accurate results.

Additional details and impacted files
@@          Coverage Diff           @@
##             main   #6072   +/-   ##
======================================
  Coverage        ?   69.0%           
======================================
  Files           ?     195           
  Lines           ?   18903           
  Branches        ?       0           
======================================
  Hits            ?   13055           
  Misses          ?    4922           
  Partials        ?     926           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rainest
Copy link
Contributor Author

rainest commented May 22, 2024

Isolated failure is

2024-05-22T22:23:02.6714216Z     tcproute_test.go:259: 
2024-05-22T22:23:02.6719081Z         	Error Trace:	/home/runner/work/kubernetes-ingress-controller/kubernetes-ingress-controller/test/integration/isolated/assertion_helpers.go:43
2024-05-22T22:23:02.6722475Z         	            				/home/runner/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.22.2.linux-amd64/src/runtime/asm_amd64.s:1695
2024-05-22T22:23:02.6723914Z         	Error:      	Received unexpected error:
2024-05-22T22:23:02.6725770Z         	            	read tcp 172.18.0.1:54756->172.18.128.3:8888: read: connection reset by peer
2024-05-22T22:23:02.6726834Z     tcproute_test.go:259: 
2024-05-22T22:23:02.6729735Z         	Error Trace:	/home/runner/work/kubernetes-ingress-controller/kubernetes-ingress-controller/test/integration/isolated/assertion_helpers.go:42
2024-05-22T22:23:02.6733557Z         	            				/home/runner/work/kubernetes-ingress-controller/kubernetes-ingress-controller/test/integration/isolated/tcproute_test.go:259
2024-05-22T22:23:02.6736910Z         	            				/home/runner/go/pkg/mod/sigs.k8s.io/e2e-framework@v0.3.1-0.20231113122213-262cac32d35e/pkg/env/env.go:435
2024-05-22T22:23:02.6739902Z         	            				/home/runner/go/pkg/mod/sigs.k8s.io/e2e-framework@v0.3.1-0.20231113122213-262cac32d35e/pkg/env/env.go:468
2024-05-22T22:23:02.6741283Z         	Error:      	Condition never satisfied
2024-05-22T22:23:02.6751201Z         	Test:       	TestTCPRouteEssentials/essentials/verifying_behavior_when_Gateway_is_deleted_and_recreated
2024-05-22T22:23:02.6753344Z     tcproute_test.go:261: deleting the Gateway

Looks like garbage from some task that has no retry attempt encountering a transient failure. Unlikely to be related to consumer group changes upstream.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge let the author merge this, don't merge for them. size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants