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

Merge helm template plugin into core Helm #1887

Closed
kfox1111 opened this issue Jan 30, 2017 · 4 comments
Closed

Merge helm template plugin into core Helm #1887

kfox1111 opened this issue Jan 30, 2017 · 4 comments

Comments

@kfox1111
Copy link

I'm trying to add helm package rendering validation to some of our lightweight gate jobs. Finding the following issues:

  • helm install xxx --dry-run --debug produces the yaml to be loaded, but also some non yaml at the beginning that is nontrivial to parse out. There maybe needs to be another flag to say, "only give me the yaml that will be uploaded"
  • helm install xxx --dry-run --debug requires a working k8s/tiller. This is a heavy lift for a lightweight validation job
  • helm template plugin. currently requires to be built from source. this is painful from a lightweight validation job
  • helm template plugin also runs the risk of being out of sync with the helm intended to be used.

Ideally, I think as an operator, I'd like to see the helm template plugin merged into the main helm client binary. Being able to see what its going to do is a very important piece of debugging functionality for operators. Would that be possible? I don't think it would add much code to the helm cli?

@technosophos technosophos changed the title template rendering Merge helm template plugin into core Helm Feb 14, 2017
@technosophos
Copy link
Member

I could see doing this, since it actually is only a tiny bit of new code. I believe this would also be fairly easy to do, so I am marking this as a starter.

@technosophos technosophos added this to the 2.4.0-Triage milestone Feb 14, 2017
@chancez
Copy link

chancez commented Jun 19, 2017

@technosophos I would love to see this. I'm basically begging someone to do this in slack 💯

@bonhamjp
Copy link

bonhamjp commented Sep 5, 2017

This issue seems to be a duplicate of #2755 which has an open PR. Should this issue be closed?

I did start working on this, not knowing it was a duplicate. I'm thinking I should have asked before starting in on it, but since it is still open someone else might get confused.

@bacongobbler
Copy link
Member

Yes, let's close this duplicate ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants