Skip to content

Support helm's new TakeOwnership sdk value - #1631

Open
atykhyy wants to merge 1 commit into
hashicorp:mainfrom
atykhyy:take-ownership
Open

Support helm's new TakeOwnership sdk value#1631
atykhyy wants to merge 1 commit into
hashicorp:mainfrom
atykhyy:take-ownership

Conversation

@atykhyy

@atykhyy atykhyy commented Jun 13, 2025

Copy link
Copy Markdown

Description

The TakeOwnership option in Helm allows Helm to adopt resources that are currently owned by other helm charts or not owned at all. It's only available through the SDK right now. This PR adds a take_ownership option to helm_release that makes use of this SDK feature. #1597 is the older draft PR by @lizduty for this feature which got stuck in draft for some reason.

Acceptance tests

  • Have you added an acceptance test for the functionality being added?

Release Note

Release note for CHANGELOG:

Adds the `take_ownership` option to `helm_release` resource.

References

#1597, helm/helm#12876

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

@atykhyy
atykhyy requested a review from a team as a code owner June 13, 2025 08:21
@hashicorp-cla-app

hashicorp-cla-app Bot commented Jun 13, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@Syndlex

Syndlex commented Jul 11, 2025

Copy link
Copy Markdown

@vravind1 can someone of your team take a look at this? It would be great to have this feature!

@antoine-ansari

Copy link
Copy Markdown

Hey, this small change can ease lots of things. Will this be reviewed soon please ?

@Syndlex

Syndlex commented Oct 2, 2025

Copy link
Copy Markdown

It looks like they merged it from somewhere else:
b278d36

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants