-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
🌱 Upgrade e2e test for latest Kubernetes versions #2371
🌱 Upgrade e2e test for latest Kubernetes versions #2371
Conversation
73e0d41
to
093f229
Compare
093f229
to
83e2ef4
Compare
83e2ef4
to
a4ee956
Compare
b3d95a7
to
51f887d
Compare
Thanks for the reviews and feedback @camilamacedo86 @estroz. Updated and ready for review again. |
51f887d
to
222bf2a
Compare
Pushed prow kubernetes/test-infra#23935 as I inadvertently set the k8s versions to the latest versions rather than the lates Kind k8s versions supported. This should fix the CI issues. |
222bf2a
to
3d427cc
Compare
Signed-off-by: Martin Hickey <[email protected]>
3d427cc
to
ea0cd76
Compare
/retest |
1 similar comment
/retest |
This can't be merged until kubernetes/test-infra#23950 merges. We need to test older versions. @hickeyma please make sure someone from the kubebuilder config OWNERS file has signed off on infra PR's in the future. Not a big deal in the end, stuff happens! |
/retest |
@hickeyma: The following tests failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Sorry my bad. It just happened to be merged very quickly. |
/retest-required |
kind v0.11.0+ supports Kubernetes 1.20+ Signed-off-by: Martin Hickey <[email protected]>
There were build issues with prow for Kubernetes v1.20+. This is because Kind v0.7.0 does not support Kubernetes v1.20+. Thanks to @BenTheElder for the "heads up" on slack. This did not show when testing locally as had v0.11.1 already installed. Pushed change to common script to install v0.11.1. CI now running as expected with all Kubernetes versions. |
@camilamacedo86 @estroz This is good to review again when you get a chance. |
@hickeyma thanks again! /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: estroz, hickeyma The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Thanks @camilamacedo86 and @estroz |
Upgrade the e2e tests for Kubernetes 1.20, 1.21 and 1.22
Closes #2306