-
-
Notifications
You must be signed in to change notification settings - Fork 66
Added host and database_name to Kubernetes secret #51
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
Conversation
Are you unable to use the |
That doesn't seem to work: |
Can you prepend |
Thanks for the suggestion, but that doesn't help: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't really have a problem with this approach then.
@hitman99 What do you think?
"PASSWORD": []byte(password), | ||
"LOGIN": []byte(login), | ||
"POSTGRES_URL": []byte(pgUserUrl), | ||
"HOST": []byte(r.pgHost), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix indentation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs proper indentations as @arnarg said
postgres-operator 0.4.3 was released with these changes: https://github.com/movetokube/postgres-operator/releases/tag/0.4.3 |
I need the host and database_name separately in my application, so I added them to the secret.