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

Add documentation for multiline expansion #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yissachar
Copy link
Contributor

No description provided.

@justinsb
Copy link
Contributor

I'm wondering if this is actually a bug in kexpand... i.e. are we painting ourselves into a corner by explicitly supporting this.

I like the idea of making the decision through docs :-)

I opened kubernetes/kubernetes#36750 to get some clarity on whether this is possible in the official syntax, and hopefully whether anyone is working on this!

Given some of the limitations, I think I would prefer to use some sort of combination logic (like kubectl apply does - they call it a smart merge or similar), rather than rely on correct indentation. But let's see what kubernetes/kubernetes#36750 flushes out.

@yissachar
Copy link
Contributor Author

Yeah, this was a kind of hacky solution to a real problem I had, but I figured I would document it so that others in the same situation could use it as well. To be honest, I'm not sure that this is considered in scope for templates; it might be that when you have these sorts of requirements it's time to move to Helm, which has much more robust template support.

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

Successfully merging this pull request may close these issues.

2 participants