-
Notifications
You must be signed in to change notification settings - Fork 84
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
[release-4.17] OCPBUGS-38483: Bump to k8s.io v0.30.3, controller-runtime v0.18.4 #422
base: release-4.17
Are you sure you want to change the base?
[release-4.17] OCPBUGS-38483: Bump to k8s.io v0.30.3, controller-runtime v0.18.4 #422
Conversation
Bump the vendored k8s.io/* packages to v0.30.3; bump the vendored sigs.k8s.io/controller-runtime package to v0.18.4, the latest release, which uses the k8s.io/* v0.30.1 packages; and bump the vendored github.com/openshift/coredns-ocp-dnsnameresolver/operator package to commit 27d076fec9c2eb167469d5aa4a2ed8a29c2c14fc, which uses controller-runtime v0.18.4 and k8s.io/* v0.30.3. go get k8s.io/[email protected] k8s.io/[email protected] k8s.io/[email protected] k8s.io/[email protected] sigs.k8s.io/[email protected] github.com/openshift/coredns-ocp-dnsnameresolver/operator@27d076fec9c2eb167469d5aa4a2ed8a29c2c14fc go mod tidy go mod vendor Update calls to controller.Watch to adapt to breaking changes in controller-runtime. Namely, the handler and predicate arguments have been removed from Watch and are now specified as options on the source. Additionally, the source.Kind type now has a type parameter, so it is necessary to specify a client.Object type argument. This commit is related to OCPBUGS-38483. https://issues.redhat.com/browse/OCPBUGS-38483 * go.mod: Bump. * go.sum: Regenerate. * pkg/operator/controller/controller.go (New): * pkg/operator/controller/status/controller.go (New): Update for the new controller.Watch and source.Kind type signatures. * vendor/*: Regenerate.
@Miciah: This pull request references Jira Issue OCPBUGS-38483, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
@Miciah: This pull request references Jira Issue OCPBUGS-38483, which is invalid:
Comment In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
/jira refresh |
@Miciah: This pull request references Jira Issue OCPBUGS-38483, which is valid. 7 validation(s) were run on this bug
Requesting review from QA contact: In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/assign @rfredette |
@Miciah: The following tests failed, say
Full PR test history. Your PR dashboard. 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-sigs/prow repository. I understand the commands that are listed here. |
/retest |
Bump the vendored k8s.io/* packages to v0.30.3; bump the vendored sigs.k8s.io/controller-runtime package to v0.18.4, the latest release, which uses the k8s.io/* v0.30.1 packages; and bump the vendored github.com/openshift/coredns-ocp-dnsnameresolver/operator package to commit 27d076fec9c2eb167469d5aa4a2ed8a29c2c14fc, which uses controller-runtime v0.18.4 and k8s.io/* v0.30.3.
Update calls to
controller.Watch
to adapt to breaking changes in controller-runtime. Namely, the handler and predicate arguments have been removed fromWatch
and are now specified as options on the source. Additionally, thesource.Kind
type now has a type parameter, so it is necessary to specify aclient.Object
type argument.This is manual backport of #420. For the release-4.17 branch of cluster-dns-operator, we need to use the commit from the corresponding branch of coredns-ocp-dnsnameresolver.