-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Bump controller-runtime to 0.7.0 #4306
Bump controller-runtime to 0.7.0 #4306
Conversation
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
go.mod
Outdated
@@ -32,7 +32,7 @@ require ( | |||
k8s.io/client-go v0.19.4 | |||
k8s.io/kubectl v0.19.4 | |||
rsc.io/letsencrypt v0.0.3 // indirect | |||
sigs.k8s.io/controller-runtime v0.7.0-alpha.7 | |||
sigs.k8s.io/controller-runtime v0.7.0 | |||
sigs.k8s.io/controller-tools v0.3.0 |
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.
sigs.k8s.io/controller-tools v0.3.0 | |
sigs.k8s.io/controller-tools v0.4.1 |
@varshaprasad96 there are a bunch more changes needed that #4294 takes care of. You can either copy them over here or close this in favor of that one. Thoughts? |
782c668
to
8bfe143
Compare
New changes are detected. LGTM label has been removed. |
* Bump controller-tools to 0.4.1 * Bump operator-framework/api to 0.4.0
8bfe143
to
c3794ab
Compare
I can close this one, since this is just bumping the versions in go.mod. |
Thanks @estroz ! |
Description of the change:
Bump controller-runtime to 0.7.0
Motivation for the change:
Checklist
If the pull request includes user-facing changes, extra documentation is required:
changelog/fragments
(seechangelog/fragments/00-template.yaml
)website/content/en/docs