-
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
⚠️ remove kustomize/v1, go/v2 and go/v3, and configurations for Project Config v2 related to legacy Kubebuilder CLI version < 3 #3763
Conversation
/hold |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: camilamacedo86 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 |
3781579
to
0ef3140
Compare
/hold cancel |
Hi @varshaprasad96 and @everettraven I will wait until next week for us start to removing all stuff Because changes in the API, it must be merged manually, so if you are able to look at and approve this one that would be great and we will only move forward anyway manually from next week |
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.
Took a first quick pass it looks good. I'll take a deeper look again.
0ef3140
to
70795dd
Compare
70795dd
to
ea1b8d5
Compare
Thank you a lot. Let's to do a patch release for what was fixed so far and then we can start to get this one merged and start to work in our MAJOR version bump Kubebuilder v4 as we have speaking so far where we have all cleaned up. :-) /hold |
ea1b8d5
to
17b0ea8
Compare
17b0ea8
to
c76531d
Compare
c76531d
to
6417d41
Compare
@camilamacedo86: 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. 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. |
6417d41
to
7b6f433
Compare
…ig v2 from legacy Kubebuilder CLI version < 3
7b6f433
to
79a3863
Compare
I am moving within since We need to merge this one in order to be able to add support for k8s 1.30 and go 1.22 |
5a7e207
into
kubernetes-sigs:master
Description
Remove deprecated plugins and doc sections as specific related tests.
Motivation
Maintainability and better usability
More info: #3622
(began to prepare the master branch for a V4 release)