chore(refactor): migrate project from Kubebuilder v3 to v4 layout #84
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it:
Inspired by
followed the official manual too.
This PR does not modify the API surface of KGO since it's currently used as a dependency in the enterprise built. Such efforts can be addressed in a separate PR.
Which issue this PR fixes
Part of #130
Special notes for your reviewer:
The layout proposed by
kubebuilder
to movecontrollers
->internal/controller
cannot be implemented because they are pulled by KGO EE, so only renaming tocontroller
was performed.Known limitation, calling e.g.
will put a newly created controller in
internal/controller
. Maybe all controllers that are not pulled explicitly should be put there... It's something to consider and write down as a separate issue.The usage of
sed
ingenerate.clientsets
is not great but I tested it with bot BSD and GNUsed
so I think we can live with it.