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 support for HelmChart as chartRef #945

Merged
merged 3 commits into from
Apr 23, 2024
Merged

Add support for HelmChart as chartRef #945

merged 3 commits into from
Apr 23, 2024

Conversation

souleb
Copy link
Member

@souleb souleb commented Apr 18, 2024

If implemented, user will be able to share an existing HelmChart custom resource between HelmReleases.

Closes: #204

Followup: #905

If implemented, user will be able to share an existing HelmChart custom
resource between HelmReleases.

Signed-off-by: Soule BA <bah.soule@gmail.com>
@stefanprodan stefanprodan added the enhancement New feature or request label Apr 18, 2024
Signed-off-by: Soule BA <bah.soule@gmail.com>
@souleb
Copy link
Member Author

souleb commented Apr 22, 2024

switching from chart template to spec.chartRef of type HelmChart

Status:
  Conditions:
    Last Transition Time:  2024-04-22T09:39:21Z
    Message:               Helm upgrade succeeded for release default/podinfo-template.v2 with chart podinfo@0.1.0
    Observed Generation:   3
    Reason:                UpgradeSucceeded
    Status:                True
    Type:                  Ready
    Last Transition Time:  2024-04-18T09:13:16Z
    Message:               Helm upgrade succeeded for release default/podinfo-template.v2 with chart podinfo@0.1.0
    Observed Generation:   2
    Reason:                UpgradeSucceeded
    Status:                True
    Type:                  Released
  History:
    Chart Name:                   podinfo
    Chart Version:                0.1.0
    Config Digest:                sha256:e15c415d62760896bd8bec192a44c5716dc224db9e0fc609b9ac14718f8f9e56
    Digest:                       sha256:60770b7d040612283cc8931f8b67fd34d446d3b5316949e7a8803de05b448599
    First Deployed:               2024-04-16T08:28:52Z
    Last Deployed:                2024-04-18T09:13:12Z
    Name:                         podinfo-template
    Namespace:                    default
    Status:                       deployed
    Version:                      2
    Chart Name:                   podinfo
    Chart Version:                6.0.4
    Config Digest:                sha256:e15c415d62760896bd8bec192a44c5716dc224db9e0fc609b9ac14718f8f9e56
    Digest:                       sha256:912442ff9c2798eec85cdd81cc5dc5bab63d952fefbf423b32d52dda5cfb426a
    First Deployed:               2024-04-16T08:28:52Z
    Last Deployed:                2024-04-16T08:28:52Z
    Name:                         podinfo-template
    Namespace:                    default
    Status:                       superseded
    Version:                      1
  Last Applied Revision:          0.1.0
  Last Attempted Config Digest:   sha256:e15c415d62760896bd8bec192a44c5716dc224db9e0fc609b9ac14718f8f9e56
  Last Attempted Generation:      3
  Last Attempted Release Action:  upgrade
  Last Attempted Revision:        0.1.0
  Observed Generation:            3
  Storage Namespace:              default
Events:
  Type    Reason            Age   From             Message
  ----    ------            ----  ----             -------
  Normal  HelmChartDeleted  23s   helm-controller  deleted HelmChart 'default/default-podinfo-template'

It successfully garbage collect the old helmChart, performs and upgrade, and keep the same release version.

@stefanprodan stefanprodan changed the title Enable HelmChart ChartRef Add support for HelmChart as chartRef Apr 22, 2024
Co-authored-by: Stefan Prodan <stefan.prodan@gmail.com>
Signed-off-by: souleb <bah.soule@gmail.com>
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thanks @souleb 🏅

@souleb souleb merged commit 9f82dde into main Apr 23, 2024
6 checks passed
@souleb souleb deleted the enable-hc-sources branch April 23, 2024 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use existing HelmChart resource in a HelmRelease
2 participants