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

Accept root credentials from existing secret #119

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jennydaman
Copy link
Contributor

Adds a feature so that the environment variables ZO_ROOT_USER_EMAIL and ZO_ROOT_USER_PASSWORD can be sourced from an existing secret.

Example values.yaml:

auth:
  existingRootUserSecret:
    name: my-openobserve-root-user

Furthermore, I fixed some minor Helm YAML gotchas: instead of "{{ .Values.auth.ZO_ROOT_USER_PASSWORD }}" the code should be {{ .Values.auth.ZO_ROOT_USER_PASSWORD | quote }} to properly handle values which contain \ or " characters.

@jennydaman
Copy link
Contributor Author

Here's how you can try it out:

kubectl create secret generic my-openobserve-root-user --from-literal=ZO_ROOT_USER_EMAIL='[email protected]' --from-literal=ZO_ROOT_USER_PASSWORD='ADifferentPass123'
helm install --generate-name --set auth.existingRootUserSecret.name=my-openobserve-root-user ./charts/openobserve-standalone

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.

1 participant