Skip to content
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

add default provisioner to helm chart #786

Merged
merged 2 commits into from
Nov 11, 2021

Conversation

bwagner5
Copy link
Contributor

@bwagner5 bwagner5 commented Nov 9, 2021

1. Issue, if available:
#781
#784

2. Description of changes:

  • Add default provisioner to helm chart
  • Move CRD to crds dir per helm3 guidance so that CRD is sequenced before the provisioner

3. Does this change impact docs?

  • Yes, PR includes docs updates
  • Yes, issue opened: link to issue
  • No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@netlify
Copy link

netlify bot commented Nov 9, 2021

✔️ Deploy Preview for karpenter-docs-prod canceled.

🔨 Explore the source changes: 0b65f71

🔍 Inspect the deploy log: https://app.netlify.com/sites/karpenter-docs-prod/deploys/618c5d308cc2e10008c74ea0

@bwagner5 bwagner5 requested a review from ellistarn November 9, 2021 21:02
@bwagner5
Copy link
Contributor Author

There seems to be an uninstall problem with helm and using CRDs. I'll have to dig into that more, but wanted to note this issue in the PR.

@ellistarn
Copy link
Contributor

Check out #781

@bwagner5 bwagner5 force-pushed the helm-default-provisioner branch from eb198a2 to 0b65f71 Compare November 11, 2021 00:00
@bwagner5 bwagner5 merged commit 42e1c20 into aws:main Nov 11, 2021
@bwagner5 bwagner5 deleted the helm-default-provisioner branch November 11, 2021 00:31
@krisdock
Copy link

krisdock commented Aug 31, 2022

@bwagner5 did this ever make it into a release? the template file and values block for this don't seem to exist in any tag other than 0.4.3.

@bwagner5
Copy link
Contributor Author

It did and then it was removed due to issues with Helm managing custom resources well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants