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

values.yaml has wrong types defined for env + envRaw #89

Open
JVMartin opened this issue Dec 15, 2021 · 2 comments
Open

values.yaml has wrong types defined for env + envRaw #89

JVMartin opened this issue Dec 15, 2021 · 2 comments
Labels
bug Something isn't working

Comments

@JVMartin
Copy link

The values.yaml file has:


However, these need to be arrays:

env: []
envRaw: []

They are unusable as objects.

e.g., with these values:

graylog:
  envRaw:
    - name: TEST
      value: test

We get the errors:

cannot overwrite table with non table for envRaw (map[])"
@JVMartin JVMartin added the bug Something isn't working label Dec 15, 2021
@tamirhad
Copy link

tamirhad commented Jan 9, 2022

creating the env vars like so works for me:

env:
          GRAYLOG_OUTPUT_BATCH_SIZE: 8000
          GRAYLOG_OUTPUTBUFFER_PROCESSORS: 6
          GRAYLOG_PROCESSBUFFER_PROCESSORS: 10

In my opinion list is more accurate, but its work anyway :)

@matthiasguentert
Copy link

creating the env vars like so works for me:

env:
          GRAYLOG_OUTPUT_BATCH_SIZE: 8000
          GRAYLOG_OUTPUTBUFFER_PROCESSORS: 6
          GRAYLOG_PROCESSBUFFER_PROCESSORS: 10

In my opinion list is more accurate, but its work anyway :)

This might be true for static environment variable definitions, but won't work with dynamic ones, e.g. like so:

env:
- name: POD_NAME
  valueFrom:
    fieldRef:
      fieldPath: metadata.name

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants