-
Notifications
You must be signed in to change notification settings - Fork 1.8k
updating ansible plugin to v1.42.0 and corresponding dependencies #7010
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
Conversation
| k8s.io/utils v0.0.0-20251002143259-bc988d571ff4 | ||
| sigs.k8s.io/controller-runtime v0.21.0 | ||
| sigs.k8s.io/controller-tools v0.18.0 | ||
| sigs.k8s.io/kubebuilder/v4 v4.6.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.
I think we need to update as well.
But it might be a second follow up
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.
I don't plan to update kubebuilder before doing another release. I think there are breaking (scaffolding) changes in versions after v4.6.0 that I don't have the bandwidth to look at right now.
The main thing is to get a release out with some CVE fixes in it.
camilamacedo86
left a comment
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
Signed-off-by: Adam D. Cornett <adc@redhat.com>
26b5bd7 to
6cf7c28
Compare
|
New changes are detected. LGTM label has been removed. |
Description of the change:
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