Skip to content

Allow keeping stdout/stderr separated when rendering helm templates #986

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

Closed
cgirard-mir opened this issue Sep 8, 2021 · 1 comment · Fixed by #1526
Closed

Allow keeping stdout/stderr separated when rendering helm templates #986

cgirard-mir opened this issue Sep 8, 2021 · 1 comment · Fixed by #1526
Labels
enhancement New feature or request trial project Suitable for trial projects

Comments

@cgirard-mir
Copy link

We have some use cases where rendering helm charts with overridden values causes helm to produce warnings on stderr, but otherwise finishes successfully. We use the helm.RenderTemplate() method to perform this rendering. Unfortunately, the method only supports returning the combined stdout+stderr and this causes our subsequent UnmarshalK8SYaml() call to fail because the text is no longer valid YAML.

It would be nice if the method could be called while keeping the two output streams separated. A couple of ideas are to either create a second method that works this way, or perhaps passing an option to the current method to allow this behavior.

@brikis98 brikis98 added enhancement New feature or request help wanted labels Sep 9, 2021
@brikis98
Copy link
Member

brikis98 commented Sep 9, 2021

That's a good point. Is returning stderr ever useful with the RenderTemplate method? Or is that just a bug, and only stdout should be returned? A PR to improve/fix this is very welcome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request trial project Suitable for trial projects
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants