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

Update helm-controller #5263

Merged
merged 1 commit into from
Mar 17, 2022
Merged

Conversation

brandond
Copy link
Member

@brandond brandond commented Mar 15, 2022

Proposed Changes

Update helm-controller

Waiting on merge of k3s-io/helm-controller#137

Types of Changes

  • Add repoCA to helmChart.spec
  • Add failurePolicy to helmChart.spec
  • Add support for helmcharts.helm.cattle.io/unmanaged annotation to unmanage a chart

Verification

See k3s-io/helm-controller#137

Linked Issues

User-Facing Change

The embedded Helm controller can now cease management of existing HelmChart releases, supports setting a failure policy for  install/update operations, and allows trusting custom CA certs for remote chart repositories.

Further Comments

cc @Martin-Weiss

@brandond brandond requested a review from a team as a code owner March 15, 2022 21:53
@brandond brandond force-pushed the bump_helm_controller branch from 7e77093 to cda126a Compare March 17, 2022 00:23
Signed-off-by: Brad Davidson <brad.davidson@rancher.com>
@brandond brandond force-pushed the bump_helm_controller branch from cda126a to 8129c11 Compare March 17, 2022 00:23
@brandond brandond merged commit a93b9b6 into k3s-io:master Mar 17, 2022
@Martin-Weiss
Copy link

@brandond - great and thanks a lot!! Any idea in which RKE2 releases we can expect this to be included? Will there be a documentation on how to use these features?

@brandond
Copy link
Member Author

I'm going to try to get it into the RKE2 releases occurring early next week, for this week's upstream patch releases. The only docs are currently in the above-linked helm-controller PR.

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.

3 participants