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
fix: use official controller-gen #4171
Conversation
Signed-off-by: Charles-Edouard Brétéché <charled.breteche@gmail.com>
@@ -10,7 +10,7 @@ GIT_BRANCH := $(shell git branch | grep \* | cut -d ' ' -f2) | |||
GIT_HASH := $(GIT_BRANCH)/$(shell git log -1 --pretty=format:"%H") | |||
TIMESTAMP := $(shell date '+%Y-%m-%d_%I:%M:%S%p') | |||
CONTROLLER_GEN=controller-gen | |||
CONTROLLER_GEN_REQ_VERSION := v0.8.0 | |||
CONTROLLER_GEN_REQ_VERSION := v0.9.1-0.20220629131006-1878064c4cdf |
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.
Looks like changes has not included/cherry-picked in latest release 0.9.2 ?
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.
Nope, not yet :(
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.
@prateekpandey14 I need to cherry pick my changes so that they are included in the next release.
Until then it's still better to use the official repo than my fork IMHO.
This PR switches back to the official controller-gen repo now kubernetes-sigs/controller-tools#683 was merged.