-
Notifications
You must be signed in to change notification settings - Fork 833
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
chore: Separated webhooks from karpenter-core #2754
Conversation
✅ Deploy Preview for karpenter-docs-prod canceled.
|
6bda3ae
to
5cb040a
Compare
61c40ff
to
3854f56
Compare
4b9b9dc
to
892a9a1
Compare
@@ -103,8 +102,13 @@ delete: ## Delete the controller from your ~/.kube/config cluster | |||
helm uninstall karpenter --namespace karpenter | |||
|
|||
codegen: ## Generate code. | |||
curl https://raw.githubusercontent.com/aws/karpenter-core/v0.0.1/chart/crds/karpenter.sh_provisioners.yaml > charts/karpenter/crds/karpenter.sh_provisioners.yaml | |||
controller-gen \ |
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.
Somehow this got lost from karpenter's release processes 🤔
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.
Good catch!
@@ -0,0 +1,70 @@ | |||
apiVersion: rbac.authorization.k8s.io/v1 |
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.
karpenter and karpenter-core charts will be split/renamed in a followon.
b5f0a31
to
1c2dcb7
Compare
6fe7cb0
to
5ab0a07
Compare
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.
Couple comments, otherwise LGTM
bc2fa61
to
a8e164a
Compare
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 🚀
Fixes #
Description
How was this change tested?
Does this change impact docs?
Release Note
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.