feat(tests): add checks for terway daemonset name and version requirements in tests#940
Conversation
…ments in tests Signed-off-by: l1b0k <libokang.lbk@alibaba-inc.com>
There was a problem hiding this comment.
This PR is being reviewed by Cursor Bugbot
Details
Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
Codecov Report✅ All modified and coverable lines are covered by tests. ❌ Your project status has failed because you have indirect coverage changes. Learn more about Unexpected Coverage Changes and reasons for indirect coverage changes. Additional details and impacted files@@ Coverage Diff @@
## main #940 +/- ##
==========================================
- Coverage 61.77% 61.74% -0.03%
==========================================
Files 131 131
Lines 16104 16104
==========================================
- Hits 9948 9944 -4
- Misses 4984 4988 +4
Partials 1172 1172
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Note
Add daemonset/name and version checks to e2e tests (incl. k8s version for HostPort) and introduce Makefile targets to run e2e suites.
terwaydaemonset name (terway-eniip) and minimum versions.tests/utils_test.go:GetTerwayDaemonSetName,GetTerwayVersion,GetK8sVersion,CheckTerwayDaemonSetName,CheckTerwayVersion.tests/config_test.go(IPPool): requireterway-eniip,terway >= v1.16.1.tests/idle_ip_reclaim_test.go: requireterway-eniip,terway >= v1.16.1; cleanup fixes.tests/connective_test.go:terway-eniip.terway-eniip,terway >= v1.15.0,k8s >= v1.34.0; minor err var fix.tests/multi_network_test.go: requireterway-eniip,terway >= v1.16.1.tests/main_test.go: minor cleanup of unused flags/vars.e2e-test,e2e-upgrade-test,e2e-test-allto run e2e suites.Written by Cursor Bugbot for commit 7f5b9c7. This will update automatically on new commits. Configure here.