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

HELM_HOME environment variable is ignored #854

Closed
markphillips100 opened this issue Oct 18, 2019 · 5 comments · Fixed by #855
Closed

HELM_HOME environment variable is ignored #854

markphillips100 opened this issue Oct 18, 2019 · 5 comments · Fixed by #855
Assignees
Labels
area/helm customer/feedback Feedback from customers impact/usability Something that impacts users' ability to use the product easily and intuitively
Milestone

Comments

@markphillips100
Copy link

Describe the bug
Running a Pulumi preview command using the task, even after a HelmInstaller task successfully runs, fails to fetch a helm chart if a Chart resource exists in the Pulumi project. The Helm tool cannot locate the .helm folder.

To Reproduce
Use a Chart resource from a Pulumi project run by the Pulumi@1 task.

Expected behavior
The helm Chart resource provider should locate the .helm folder using the HELM_HOME environment variable.

Screenshots
pulumi-helm-preview-error

Additional context
A demo exists here with a pipeline already setup and failing. The failing build log is here.

@lblackstone lblackstone transferred this issue from pulumi/pulumi-az-pipelines-task Oct 21, 2019
@lblackstone lblackstone added area/helm customer/feedback Feedback from customers impact/usability Something that impacts users' ability to use the product easily and intuitively labels Oct 21, 2019
@lblackstone lblackstone added this to the 0.28 milestone Oct 21, 2019
@lblackstone lblackstone changed the title Helm chart fails preview HELM_HOME environment variable is ignored Oct 21, 2019
@lblackstone
Copy link
Member

@markphillips100 Thanks for the detailed bug report! I've fixed the issue in the linked PR, so this should work as you expect in the next release.

@markphillips100
Copy link
Author

@lblackstone you are welcome, and thanks for fixing it so fast! I'll test after the release.

@lblackstone
Copy link
Member

@markphillips100 Sorry for the delay in getting this merged; things were a little hectic leading up to KubeCon. I'll cut a release that includes this fix later this week, or you can try it out using the dev tag prior to that.

@markphillips100
Copy link
Author

@lblackstone I should be able to test next week so will give you an update shortly after. Thanks.

@markphillips100
Copy link
Author

@lblackstone just an update to say I've tested this and it works well. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/helm customer/feedback Feedback from customers impact/usability Something that impacts users' ability to use the product easily and intuitively
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants