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

AWS EBS CSI Driver Helm chart to inject environment variables #722

Closed
barric1 opened this issue Feb 4, 2021 · 4 comments · Fixed by #856
Closed

AWS EBS CSI Driver Helm chart to inject environment variables #722

barric1 opened this issue Feb 4, 2021 · 4 comments · Fixed by #856
Labels
feature New feature to implement

Comments

@barric1
Copy link

barric1 commented Feb 4, 2021

We’d like the AWS EBS CSI Driver Helm chart to have an option to inject environment variables to pods, so that we can set AWS_STS_REGIONAL_ENDPOINTS=regional and use the Helm chart as-is without patching it after the deployment to a private VPC.

Alternatives considered aren't considered workable for deployments at scale as we want to deploy at scale and keep our configurations as simple as possible.

@wongma7 wongma7 added the feature New feature to implement label Feb 4, 2021
@georambo
Copy link

I need this as well. ebs-plugin only works for me when we make the calls through an proxy, so need to drop in

ENV:
  - name: https_proxy
    value: xxxxxx

This is pretty common feature in most charts though, to drop in additional environment variables.

@ayberk
Copy link
Contributor

ayberk commented Feb 25, 2021

related #765

@ayberk
Copy link
Contributor

ayberk commented Apr 13, 2021

/close

see #817

@k8s-ci-robot
Copy link
Contributor

@ayberk: Closing this issue.

In response to this:

/close

see #817

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

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

Successfully merging a pull request may close this issue.

5 participants