Skip to content
This repository was archived by the owner on Nov 9, 2025. It is now read-only.
This repository was archived by the owner on Nov 9, 2025. It is now read-only.

[Feature Request] Add environment variables #19

Open

Description

@CygnusHyoga

It would be nice to add environment variables to the helm chart.

# -- Directly set environment variables
env: []

For example:

# -- Directly set environment variables
env:
  TZ: Europe/Madrid

Helm chart reference/example: https://github.com/community-tooling/charts/tree/main/charts/generic

Thanks.

Activity

  1. psych0d0g commented on Jan 20, 2024

    @psych0d0g
    Contributor

    Thank you for the idea, i do understand that alot of charts have this functionality.
    Personally i would prefer this chart not to tho, in my opinion a structured config array is more friendly to the user, if there are certain settings / vars missing from the values file they should be added inside the romm.config structure instead.

    Doing it "static" in the chart also helps with redundant settings, there is no way for a user to add an ROMM env var to the env array that is already set to a diffrent value using the romm.config structure. This could lead to confusion.

    If you have any specific "issue" that can not be implemented in the config structure let me know and i am happy to further discuss

  2. self-assigned this
    on Jan 20, 2024
  3. CygnusHyoga commented on Jan 23, 2024

    @CygnusHyoga
    Author

    Ok, I understand the logic you mention. Could you at least add a variable in the values ​​to define the time zone? Thanks!

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

Metadata

Metadata

Assignees

Labels

wontfixThis will not be worked on

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions