-
Notifications
You must be signed in to change notification settings - Fork 599
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
Test(e2e): List entities instead of dump.Get to fetch Kong config in e2e tests #6358
Conversation
E2E (targeted) tests with KIND-based clusters were started at https://github.com/Kong/kubernetes-ingress-controller/actions/runs/10160735452 |
3401fb0
to
0e312f4
Compare
0e312f4
to
89af240
Compare
E2E (targeted) tests with KIND-based clusters were started at https://github.com/Kong/kubernetes-ingress-controller/actions/runs/10173800539 |
89af240
to
481ae58
Compare
E2E (targeted) tests with KIND-based clusters were started at https://github.com/Kong/kubernetes-ingress-controller/actions/runs/10191095350 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6358 +/- ##
======================================
Coverage ? 74.0%
======================================
Files ? 200
Lines ? 20179
Branches ? 0
======================================
Hits ? 14942
Misses ? 4244
Partials ? 993 ☔ View full report in Codecov by Sentry. |
E2E (targeted) tests with KIND-based clusters were started at https://github.com/Kong/kubernetes-ingress-controller/actions/runs/10193698357 |
What this PR does / why we need it:
List
Services
,Routes
,Upstreams
andTargets
to verify Kong configuration in gateway pods inverifyIngressWithEchoBackendsInAdminAPI
instead ofdump.Get
. This could reduce the concurrency of streams to forwarded ports.e2e run passes: https://github.com/Kong/kubernetes-ingress-controller/actions/runs/10173800539.
Which issue this PR fixes:
Fixes #6317
Special notes for your reviewer:
PR Readiness Checklist:
Complete these before marking the PR as
ready to review
:- [ ] theNo need to update changelog.CHANGELOG.md
release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR