⚠ bump to kubernetes 1.22.0#1626
Conversation
Signed-off-by: Joe Lanford <joe.lanford@gmail.com>
…nd validating webhook configs Signed-off-by: Joe Lanford <joe.lanford@gmail.com>
k8s-ci-robot
added
the
cncf-cla: yes
k8s-ci-robot
added
approved
|
@joelanford: The following test 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. DetailsInstructions 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. |
|
/hold |
k8s-ci-robot
added
the
do-not-merge/hold
varshaprasad96
left a comment
varshaprasad96
left a comment
There was a problem hiding this comment.
just one non-blocker. would be helpful to add a todo-comment to migrate the tests in controllerutil, since extensions/v1beta1 of Deployment is deprecated from 1.22
| // If both this field and CRDs field in CRDInstallOptions are specified, the | ||
| // values are merged. | ||
| CRDs []client.Object | ||
| CRDs []apiextensionsv1.CustomResourceDefinition |
There was a problem hiding this comment.
Just a doubt. apiextensionsv1.CustomResourceDefinition does seem apt, but was there any specific reason why client.Object was used previously ? (was it just because v1beta1 crds were also supported)
There was a problem hiding this comment.
was it just because v1beta1 crds were also supported
That's exactly it.
k8s-ci-robot
added
the
lgtm
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: joelanford, varshaprasad96, vincepri The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
@joelanford Feel free to unhold, changes looks good! |
|
/hold cancel |
k8s-ci-robot
removed
the
do-not-merge/hold
Environment.CRDs is now a slice of apiextensionsv1.CustomResourceDefinitions instead of client.Object kubernetes-sigs/controller-runtime#1626 (comment) Signed-off-by: irbekrm <irbekrm@gmail.com>
Environment.CRDs is now a slice of apiextensionsv1.CustomResourceDefinitions instead of client.Object kubernetes-sigs/controller-runtime#1626 (comment) Signed-off-by: irbekrm <irbekrm@gmail.com>
* Upgrade to k8s.io/*@v0.22.2 in go.mod
* [automated] make revendor
* [automated] make generate
* [automated] make revendor
github.com/go-openapi/spec seems to be orphaned after previous make generate
* Upgrade to c-r@v0.10.2 in go.mod
Also, upgrade setup-envtest (doesn't have a tagged release yet, so use
release commit instead)
* [automated] make revendor
* Upgrade to controller-tools@v0.7.0 in go.mod
* [automated] make revendor
* Add missing WarningsOn{Create,Update} to rest strategies
* Replace dot imports for github.com/onsi/gomega/types
Fix linting errors: `Assertion` redeclared in this block (typecheck)
* Switch to typed values for WebhookInstallOptions.*Webhooks
ref kubernetes-sigs/controller-runtime#1626
* RequestCertificate now takes an optional requestedDuration
ref kubernetes/kubernetes#99494
* Switch to matchers.DeepEqual to test semantic equality
Maps (e.g. labels, selectors, resource requirements) might be sorted differently
than expected. Hence, use semantic equality instead of strict equality, as this
is what matters to us.
Also, DeepEqual outputs yaml and adds a nice diff indicator instead of printing
some large confusing go struct representation.
* Add new memorySwap field to expected kubelet config
ref kubernetes/kubernetes#102823
* Round condition.lastUpdateTime to seconds in test
There were several changes in the fake clients that might cause the failure
to happen just now.
* Correct unit tests falsely succeeding
These tests were not preparing the test objects correctly: they only updated
them in memory but not on the fake client. This wasn't caught until now
because the fake client mimicked the real json decoder, which didn't unset
fields not present on the server. Now that the fake client zeroes fields,
the tests started failing (which is correct). So fix the tests.
ref kubernetes-sigs/controller-runtime#1651
* Remove workarounds for missing zeroing in json decoder
Now that the c-r client zeroes fields before decoding into the object,
we can drop our workarounds for this, so basically drop
kutil.CreateResetObjectFunc and its usages.
ref kubernetes-sigs/controller-runtime#1640
* Drop setting webhook gvk explicitly in envtest
webhookConfig.SetGroupVersionKind is not needed anymore with
kubernetes-sigs/controller-runtime#1665
* Add some follow-up TODO comments
* [automated] make generate
but with go 1.16.9
* Address review comments
4 participants
This PR bumps controller-runtime to use kubernetes 1.22 libraries.
Many important APIs are removed in 1.22, most notably:
Therefore, this PR removes all usage and support for APIs removed in kubernetes 1.22. Specifically:
apiextensionsv1.ConversionReviewandapiextensionsv1.ConversionResponseLastly, there are some Go API changes in the envtest libraries that enforce the above API removals. See the go-apidiff logs from this PR (e.g. https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_controller-runtime/1626/pull-controller-runtime-apidiff-master/1423680786028040192).