-
Notifications
You must be signed in to change notification settings - Fork 474
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
Upgrade operator-sdk to 1.23.0 #1055
Upgrade operator-sdk to 1.23.0 #1055
Conversation
Signed-off-by: Israel Blancas <iblancasa@gmail.com>
Signed-off-by: Israel Blancas <iblancasa@gmail.com>
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.
We should improve how to install the Kustomize.
Makefile
Outdated
$(call go-get-tool,$(KUSTOMIZE),sigs.k8s.io/kustomize/kustomize/v4,$(KUSTOMIZE_VERSION)) | ||
kustomize: $(KUSTOMIZE) ## Download kustomize locally if necessary. | ||
$(KUSTOMIZE): $(LOCALBIN) | ||
test -s $(LOCALBIN)/kustomize || { curl -s $(KUSTOMIZE_INSTALL_SCRIPT) | bash -s -- $(subst v,,$(KUSTOMIZE_VERSION)) $(LOCALBIN); } |
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.
Could we avoid using curl or bash to do that step?
Signed-off-by: Israel Blancas <iblancasa@gmail.com>
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.
* Upgrade operator-sdk to 1.23.0 Signed-off-by: Israel Blancas <iblancasa@gmail.com> * Upgrade generated files for operator-sdk 1.23.0 Signed-off-by: Israel Blancas <iblancasa@gmail.com> * Undo changes in kustomize installation Signed-off-by: Israel Blancas <iblancasa@gmail.com> Signed-off-by: Israel Blancas <iblancasa@gmail.com>
Upgrade
operator-sdk
version from1.22.1
to1.23.0