-
Notifications
You must be signed in to change notification settings - Fork 33
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
MF-64 - Fix db user role for postgresql-users #65
Conversation
@@ -53,6 +53,10 @@ spec: | |||
value: {{ .Release.Name }}-envoy:{{ .Values.authn.grpcPort }} | |||
- name: MF_TOKEN_RESET_ENDPOINT | |||
value: "#/auth/reset-password" | |||
- name: MF_USERS_ADMIN_EMAIL |
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.
Admin of what? Some general admin, or only PotgreSQL admin?
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.
No database. It's Mainflux default user, created on startup
https://github.com/mainflux/mainflux/blob/master/users/README.md
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.
OK, I see. Thanks.
Add new envars Signed-off-by: Ivan Milosevic <iva@blokovi.com>
Signed-off-by: Ivan Milosevic <iva@blokovi.com>
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.
LGTM
* Change postgresql-users user to postgres Add new envars Signed-off-by: Ivan Milosevic <iva@blokovi.com> * Get envars values for connection to postgres databases from chart values Signed-off-by: Ivan Milosevic <iva@blokovi.com>
Change postgresql-users user to postgres
Add new envars
Resolves #64
Signed-off-by: Ivan Milosevic iva@blokovi.com