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

Multiple charts in the same application could be handled better. #69

Open
Joibel opened this issue Oct 20, 2022 · 6 comments
Open

Multiple charts in the same application could be handled better. #69

Joibel opened this issue Oct 20, 2022 · 6 comments
Labels
enhancement New feature or request

Comments

@Joibel
Copy link
Contributor

Joibel commented Oct 20, 2022

Because both charts are rendered with the same name (ARGOCD_APP_NAME), there are issues in most cases.

Provide a way to override this on a per chart basis. Can we do something automatically perhaps, like postpend the directory name?

@Joibel
Copy link
Contributor Author

Joibel commented Oct 20, 2022

Option 1: provide chartName file in the directory with the chart to override the rendered name. Not great for application reuse perhaps though.
Option 2: postpend the directoryname automatically when this scenario is detected. Redeploys the whole chart if an app goes from one to two charts.
Option 3: control through complex environment variables
Option 4: provide chartName to postpend. Currently, I like this best.

@stale
Copy link

stale bot commented Dec 19, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Dec 19, 2022
@tico24
Copy link
Collaborator

tico24 commented Dec 19, 2022

keep alive

@stale stale bot removed the wontfix This will not be worked on label Dec 19, 2022
@stale
Copy link

stale bot commented Feb 17, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Feb 17, 2023
@Joibel Joibel added enhancement New feature or request and removed wontfix This will not be worked on labels Feb 18, 2023
@Diliz
Copy link

Diliz commented Apr 13, 2023

Is something like this would help you doing what you're trying to achieve here?
#137

@Joibel
Copy link
Contributor Author

Joibel commented Apr 13, 2023

It's an alternative strategy, but fundamentally multiple plain helm files could still use separate names without helmfile.

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

No branches or pull requests

3 participants