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

[ytt] Support an array in config_yaml attribute #12

Open
braunsonm opened this issue Feb 8, 2021 · 5 comments
Open

[ytt] Support an array in config_yaml attribute #12

braunsonm opened this issue Feb 8, 2021 · 5 comments
Labels
discussion This issue is not a bug or feature and a conversation is needed to find an appropriate resolution

Comments

@braunsonm
Copy link

braunsonm commented Feb 8, 2021

Describe the problem/challenge you have
Since on the command line with ytt you can pass in multiple config files, you should be able to use this property to pass in an array of YAML configs. This is a common usecase where the YAML is retrieved from another terraform data source and read off as a property. For instance we grab some YAML from Vault and feed it into our ytt data source.

Describe the solution you'd like
A config_yaml property that supports an array of YAML source which is added to the ytt overlay command.

Anything else you would like to add:
You can get around this issue by using a join config_yaml = join("\n\n", [yaml1, yaml2]) for now.

@braunsonm braunsonm added carvel-triage This issue has not yet been reviewed for validity enhancement This issue is a feature request labels Feb 8, 2021
@github-actions
Copy link

This issue is being marked as stale due to a long period of inactivity and will be closed in 5 days if there is no response.

@github-actions github-actions bot added the stale This issue has had no activity for a while and will be closed soon label Mar 20, 2021
@braunsonm
Copy link
Author

Bump

@github-actions github-actions bot removed the stale This issue has had no activity for a while and will be closed soon label Mar 21, 2021
@cppforlife
Copy link
Contributor

challenge here is that there is only one stdin (that's where config_yaml is piped into). im not sure how we could do this securely without introducing new feature to ytt to support getting content from other file descriptors.

@github-actions
Copy link

This issue is being marked as stale due to a long period of inactivity and will be closed in 5 days if there is no response.

@github-actions github-actions bot added the stale This issue has had no activity for a while and will be closed soon label May 13, 2021
@braunsonm
Copy link
Author

activity

@github-actions github-actions bot removed the stale This issue has had no activity for a while and will be closed soon label May 15, 2021
@cppforlife cppforlife added discussion This issue is not a bug or feature and a conversation is needed to find an appropriate resolution and removed carvel-triage This issue has not yet been reviewed for validity enhancement This issue is a feature request labels May 17, 2021
@cppforlife cppforlife changed the title Support an array in k14s_ytt config_yaml attribute [ytt] Support an array in config_yaml attribute Oct 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion This issue is not a bug or feature and a conversation is needed to find an appropriate resolution
Projects
None yet
Development

No branches or pull requests

2 participants