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

Enable OCI chart lookup for available package detail. #5986

Merged
merged 4 commits into from
Feb 13, 2023

Conversation

absoludity
Copy link
Contributor

@absoludity absoludity commented Feb 13, 2023

Signed-off-by: Michael Nelson minelson@vmware.com

Description of the change

This PR fixes #5833 by ensuring that the chart info -> name is always the Helm name used in the release (ie. that in the Chart.yaml).

This ensures that we can look up an available package detail using the info which helm has at the time (the actual chart name) while still allowing Kubeapps to use ChartID (which includes the repo and any prefix).

Benefits

We can upgrade OCI releases again.

Possible drawbacks

Applicable issues

Additional information

Signed-off-by: Michael Nelson <minelson@vmware.com>
@netlify
Copy link

netlify bot commented Feb 13, 2023

Deploy Preview for kubeapps-dev canceled.

Name Link
🔨 Latest commit aecf026
🔍 Latest deploy log https://app.netlify.com/sites/kubeapps-dev/deploys/63eabbc11f0b15000899cdbf

Signed-off-by: Michael Nelson <minelson@vmware.com>
Signed-off-by: Michael Nelson <minelson@vmware.com>
@absoludity absoludity marked this pull request as ready for review February 13, 2023 06:30
@absoludity absoludity changed the title WIP: Still fixing insertion of files Enable OCI chart lookup for available package detail. Feb 13, 2023
Signed-off-by: Michael Nelson <minelson@vmware.com>
@absoludity absoludity merged commit e970564 into main Feb 13, 2023
@absoludity absoludity deleted the 5833-cant-upgrade-oci branch February 13, 2023 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Actions (e.g. upgrade) disabled on installed OCI type helm charts from ACR
2 participants