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

extraEnv overwrites required environment variables #3

Open
rlabrecque opened this issue Nov 24, 2020 · 1 comment
Open

extraEnv overwrites required environment variables #3

rlabrecque opened this issue Nov 24, 2020 · 1 comment

Comments

@rlabrecque
Copy link

Specifying .Values.auth.extraEnv or .Values.proxy.extraEnv overwrites the default values which are required, resulting in an error.

  extraEnv:
    - name: STATSD_HOST
      value: localhost
    - name: STATSD_PORT
      value: "11111"

Would be nice if they didn't need to be re-specified when overriding.

@bartmanus
Copy link

While this project seems abandoned, along with the buzzfeed/sso upstream, I want to add a note for any future maintainer: The current master values use the buzzfeed/sso:v3.0.0 container image. The upstream v3.0.0 proxy program configuration respects variables METRICS_STATSD_HOST and METRICS_STATSD_PORT.

Farewell, archeologist!

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

2 participants