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.

Sign up for GitHub

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

Jump to bottom

Create a no-op helm.v4.Release resource #3263

Closed
mjeffryes opened this issue Oct 15, 2024 · 3 comments · Fixed by #3305
Closed

Create a no-op helm.v4.Release resource #3263

mjeffryes opened this issue Oct 15, 2024 · 3 comments · Fixed by #3305
Assignees
@EronWright
Labels
area/helm kind/enhancement Improvements or new features needs-design Needs input from design/UX resolution/fixed This issue was fixed
Milestone
0.112

Comments

@mjeffryes
Copy link
Member

As revealed in this comment, pulumi/pulumi#5961 (comment), users can misinterpret the fact that there is a v4.Chart, but not a v4.Release to mean that Release is somehow not as up-to-date as rather than just a different tool.

While we could try to solve this with by adding some disclaimers to the docs, it's probably simpler to just create bump release to helm.v4.Release and create an alias for v3.Release.
@mjeffryes mjeffryes added kind/enhancement Improvements or new features needs-triage Needs attention from the triage team labels Oct 15, 2024
@blampe
Copy link
Contributor

blampe commented Oct 22, 2024

I think it would make more sense for v4.Release to be multi-language, so not just a pointer to v3.Release. In other words I do think there is work to be done for a v4 resource.

@blampe blampe added area/helm needs-design Needs input from design/UX and removed needs-triage Needs attention from the triage team labels Oct 22, 2024
@mjeffryes mjeffryes added this to the 0.112 milestone Oct 30, 2024
@EronWright

This comment has been minimized.

Sign in to view
@EronWright
Copy link
Contributor

Upon discussion with @mjeffryes , we will use this ticket to make doc changes to clarify that Release/v3 is the 'latest' version of the resource and that it is not deprecated.

Meanwhile we'll work towards a full-featured v4 resource as outlined here:
https://github.com/orgs/pulumi/projects/223/views/2

@EronWright EronWright mentioned this issue Nov 4, 2024
Merged
@pulumi-bot pulumi-bot added the resolution/fixed This issue was fixed label Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@EronWright EronWright

Labels
area/helm kind/enhancement Improvements or new features needs-design Needs input from design/UX resolution/fixed This issue was fixed
Projects
None yet
Milestone
0.112
Development

Successfully merging a pull request may close this issue.

doc note for release/v4 pulumi/pulumi-kubernetes
4 participants
@blampe @mjeffryes @EronWright @pulumi-bot