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

docs: multiplayer env variable correction #214

Merged
merged 1 commit into from
Mar 6, 2025

Conversation

davidaparicio
Copy link
Contributor

As discussed in the issue #199, I found an error to add the environment variable as it was written ;-)

In fact, the environmentVariables map wasn't correct set

@davidaparicio
Copy link
Contributor Author

The error was cannot patch "retool-multiplayer-ws" with kind Deployment: Deployment.apps "retool-multiplayer-ws" is invalid: spec.template.spec.containers[0].env[13].name: Invalid value: "0": a valid environment variable name must consist of alphabetic characters, digits, '_', '-', or '.', and must not start with a digit (e.g. 'my.env-name', or 'MY_ENV.NAME', or 'MyEnvName1', regex used for validation is '[-._a-zA-Z][-._a-zA-Z0-9]*')

The 13th value was the mine in my values.yaml, defined as it was in the documentation :)

environmentVariables:
- name: MAIN_DOMAIN
  value: retool.foo.com

@jjlgao jjlgao merged commit 9f91d42 into tryretool:main Mar 6, 2025
9 of 11 checks passed
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

Successfully merging this pull request may close these issues.

2 participants