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

fix: init manifests fail if chart artifact name differs from chart name #2133

Merged
merged 2 commits into from
Sep 16, 2024

Conversation

rohantmp
Copy link
Contributor

@rohantmp rohantmp commented Sep 11, 2024

part of ENG-4457

verified manually that bundled charts work with and without a provided name

Notes for reviewer:

  • charts are now put in unique directories so that the artifact name (which cannot be controlled by helm pull) doesn't matter. The pulled chart will be the only file in that directory.

fixes: #2105

@rohantmp rohantmp marked this pull request as draft September 11, 2024 10:53
Copy link

netlify bot commented Sep 11, 2024

Deploy Preview for vcluster-docs canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit 723b657
🔍 Latest deploy log https://app.netlify.com/sites/vcluster-docs/deploys/66e18e8edb8262000887556b

@rohantmp rohantmp added the backport-to-v0.20 backport this PR to v0.20 branch label Sep 11, 2024
@rohantmp rohantmp force-pushed the fixHelm-4457 branch 2 times, most recently from 0925e17 to 27ce172 Compare September 11, 2024 11:52
@rohantmp rohantmp marked this pull request as ready for review September 11, 2024 12:07
removed test that no longer tested anything

Signed-off-by: Rohan CJ <rohantmp@gmail.com>
Signed-off-by: Rohan CJ <rohantmp@gmail.com>
@rohantmp
Copy link
Contributor Author

ugh, tests passed, but there was a typo in commit message. Ready to review

@FabianKramm FabianKramm merged commit eaf57a2 into loft-sh:main Sep 16, 2024
62 checks passed
@loft-bot
Copy link

💚 All backports created successfully

Status Branch Result
v0.20

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

FabianKramm added a commit that referenced this pull request Sep 16, 2024
[v0.20] Merge pull request #2133 from rohantmp/fixHelm-4457
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-v0.20 backport this PR to v0.20 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't deploy helm chart when artefact file name is different than chart name
3 participants