-
Notifications
You must be signed in to change notification settings - Fork 15
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
chore(charts): allow using existing secrets #947
Conversation
847395d
to
b4d07f6
Compare
5fd18eb
to
b08da8e
Compare
Signed-off-by: Thibault Camalon <[email protected]>
Signed-off-by: Thibault Camalon <[email protected]>
…tName Signed-off-by: Thibault Camalon <[email protected]>
Signed-off-by: Thibault Camalon <[email protected]>
Signed-off-by: Thibault Camalon <[email protected]>
Signed-off-by: Thibault Camalon <[email protected]>
Signed-off-by: Thibault Camalon <[email protected]>
Signed-off-by: Thibault Camalon <[email protected]>
Signed-off-by: Thibault Camalon <[email protected]>
Signed-off-by: Thibault Camalon <[email protected]>
Signed-off-by: Thibault Camalon <[email protected]>
Signed-off-by: Thibault Camalon <[email protected]>
Signed-off-by: Thibault Camalon <[email protected]>
Signed-off-by: Thibault Camalon <[email protected]>
Signed-off-by: Thibault Camalon <[email protected]>
b08da8e
to
1b69ac4
Compare
| Name | Description | Value | | ||
| ---------------------------------- | ------------------------------------------------------------------------------------------------------------------- | ------ | | ||
| `postgresql.enabled` | Deploy a PostgreSQL instance along the backend for its use | `true` | | ||
| `database.auth.existingSecretName` | An alternative to giving username and password; must have `OBJECTSTORE_ACCESSKEY` and `OBJECTSTORE_SECRETKEY` keys. | `""` | |
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.
This key is duplicated 🤔
{{- if .Values.redis.auth.existingSecretName -}} | ||
{{- .Values.database.auth.existingSecretName }} |
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 think the values have not been changed here
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.
🤦
*/}} | ||
{{- define "substra-backend.redis.secretName" -}} | ||
{{- if .Values.redis.auth.existingSecretName -}} | ||
{{- .Values.database.auth.existingSecretName }} |
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.
same here
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.
also, the new redis value has not been documented
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.
The redis section has:
## @skip redis
*/}} | ||
{{- define "substra-backend.accountOperator.secretName" -}} | ||
{{- if .Values.addAccountOperator.existingSecretName -}} | ||
{{- .Values.database.auth.existingSecretName }} |
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.
and here
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.
😇
Description
Part of FL-1642
existingSecretName
key for the following sectionsdatabase
addAccountOperator
redis
minio
existingSecret
key forpostgres
section (see docs)How has this been tested?
Checklist