-
Notifications
You must be signed in to change notification settings - Fork 867
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
chore: update e2e k8s versions #2637
chore: update e2e k8s versions #2637
Conversation
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #2637 +/- ##
=======================================
Coverage 81.50% 81.50%
=======================================
Files 133 133
Lines 19852 19852
=======================================
Hits 16181 16181
Misses 2836 2836
Partials 835 835 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Signed-off-by: zachaller <[email protected]>
Signed-off-by: zachaller <[email protected]>
c211c6b
to
360bf8b
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
run e2e tests on more current k8s versions, going forward 4 will be the most we support and will start dropping older versions.
I also updated the crd's from their upstreams for e2e tests.
I also fixed a bug with the tmate logic on manual runs, an empty string passes as true on actions and so we would always fall into debug mode.