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

Warn on un-supported helm hooks #1454

Closed
clstokes opened this issue Jan 28, 2021 · 0 comments · Fixed by #1460
Closed

Warn on un-supported helm hooks #1454

clstokes opened this issue Jan 28, 2021 · 0 comments · Fixed by #1460
Assignees
Labels
kind/enhancement Improvements or new features
Milestone

Comments

@clstokes
Copy link

Problem description

Pulumi's kubernetes provider does not support helm hooks (#555), but does not give the end user any warning about this. This can cause problems for helm chart deployments but in non-obvious ways.

Suggestions for a fix

Until support for helm hooks is completed, pulumi should print a message to the user on the presence of helm hooks to better guide them on issues or unexpected behavior that they get.

Something like this, but worded better:

 
Diagnostics:
  kubernetes:helm.sh/v3:Chart (my-chart):
    Note: this chart uses helm hooks which are not currently supported. This chart may not deploy successfully. See LINK for more details.
 
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Improvements or new features
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants