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

Support extra environment variables in the Chart (e.g. for HTTP Proxy) #636

Closed
outscale-age opened this issue Mar 6, 2023 · 1 comment
Closed

Comments

@outscale-age
Copy link
Contributor

Is your feature request related to a problem? Please describe.

I would like to use osc-bsu-csi-driver behind an HTTP proxy. A possible way to do this is to pass environment variables like https_proxy to the BSU CSI Driver containers. However, this is not supported today, and forces me to work around by patching the Chart.

Describe the solution you'd like in detail

Two enhancements could be done:

  • A generic way of overriding environment variables on the BSU CSI Driver (see extraEnv suggestion below).
  • Documenting the required environment variables to run the BSU CSI Driver behind a Proxy (https_proxy, no_proxy).

Additional context

Helm charts typically have knobs like extraEnv in the values.yaml to set static extra environment variables in the Deployments, StatefulSets, etc.

Some variables could contain secrets, and therefore often an extraSecretEnv is also offered, which manages a dedicated secret and its presentation.

Example overrides in a chart: https://github.com/hashicorp/vault-helm/blob/main/values.yaml#L511

@ghost
Copy link

ghost commented Sep 13, 2023

@outscale-age
HTTP Proxy is now included in osc-bsu-csi charts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants