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 volume mounts to Flagger helm chart #1687

Open
Jaxwood opened this issue Jul 20, 2024 · 0 comments
Open

Add volume mounts to Flagger helm chart #1687

Jaxwood opened this issue Jul 20, 2024 · 0 comments

Comments

@Jaxwood
Copy link

Jaxwood commented Jul 20, 2024

Describe the feature

Support for volume mounts in Flagger helm chart.

What problem are you trying to solve?

I would like to have the ability to define volumeMounts in the deployment manifest.

Currently the helm chart only supports using the additionalVolumes value in the values file.

You can't define volume mounts that uses the volumes you define.

Proposed solution

Add additionalVolumeMounts to the Flagger helm chart values file.

In the deployment template use that value when rendering the manifest.

Any alternatives you've considered?

Patch the deployment manifest using Kustomize.

Is there another way to solve this problem that isn't as good a solution?

I could use Kustomize to patch the deployment. In our case we are using Flux with HelmReleases and would prefer to keep things similar.

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

1 participant