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 a --file flag to use a different chartpress.yaml #170

Closed
rokroskar opened this issue Jul 28, 2022 · 3 comments · Fixed by #234
Closed

add a --file flag to use a different chartpress.yaml #170

rokroskar opened this issue Jul 28, 2022 · 3 comments · Fixed by #234
Labels
enhancement New feature or request

Comments

@rokroskar
Copy link
Contributor

Proposed change

In projects involving more than one chart, it would be useful to have a --file flag such that a specific chartpress.yaml file could be targeted, e.g.

chartpress --file another-chartpress.yaml

Alternative options

This behavior can be achieved by separating out charts via the directory structure and running chartpress in subdirectories, but this is sub-optimal, tedious and error-prone.

Who would use this feature?

Anyone building multiple charts in a single repo that don't all need to be built/packaged at the same cadence.

@manics
Copy link
Member

manics commented Jul 28, 2022

This is very similar to #118
Do you mind adding your suggestion on that issue instead and we can consider them together?

@rokroskar
Copy link
Contributor Author

Thanks for reminding me about that issue! They seem to be addressing pretty different things though, I'm not sure they should be wrapped into one. #118 is about relative path handling after the chartpress.yaml file has already been parsed; this request is simply to allow one to specify an alternative chartpress.yaml. But maybe you have a solution in mind for #118 that makes this request here moot?

@rokroskar
Copy link
Contributor Author

Actually, it looks like #118 has been fixed by #161 🎉

At least I no longer get an error trying to publish a chart that has the chartpress.yaml file at the root of the repo. 😆

adamblake added a commit to adamblake/chartpress that referenced this issue Mar 26, 2024
manics added a commit that referenced this issue Jun 25, 2024
Allow specifying config file with cli option (#170)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants