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

Helm: Should be able to set certain values in values.yaml to null instead of {} #443

Open
harryttd opened this issue May 10, 2022 · 0 comments

Comments

@harryttd
Copy link
Collaborator

For example:

# This would be an attempt to not create any nodes via a specifying it in a secondary values file.
# It wouldn't work bec the nodes section of the chart's default values.yaml file will get merged into
# it, creating a rolling node.
nodes: {}

# let this work instead to not create any nodes:
nodes: null

A use case would be wanting to only deploy a remote signer.

Resolving this issue will require updates in config generator, with possibly needing other updates as well. For example:

NODES = json.loads(os.environ["NODES"])

The above will fail bec it is expecting a dict for the NODES env var.

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

No branches or pull requests

2 participants