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

[HelmChart] Fix postrgres secret name for cronjob #15072

Merged
merged 1 commit into from
Nov 2, 2020

Conversation

metal3d
Copy link
Contributor

@metal3d metal3d commented Nov 1, 2020

The cronjob tries to get key from mastodon secret instead of
mastodon-postgresql - so the cronjob fails with this error:

Error: couldn't find key postgresql-password in Secret [NS]/mastodon

Another solution is to save the postgres password in mastodon secret,
but that means that the password is placed in two places.

Postgresql uses <fullname>-postgresql name as secret name.

The cronjob tries to get key from `mastodon` secret instead of
`mastodon-postgresql` - so the cronjob fails with this error:

Error: couldn't find key postgresql-password in Secret [NS]/mastodon

Another solution is to save the postgres password in mastodon secret,
but that means that the password is placed in two places.

Postgresql use <fullname>-postgresql name as secret name.
@metal3d metal3d changed the title Fix postrgres secret name for cronjob [HelmChart] Fix postrgres secret name for cronjob Nov 1, 2020
@Gargron Gargron merged commit 4b2ec4a into mastodon:master Nov 2, 2020
umonaca pushed a commit to umonaca/mastodon that referenced this pull request Nov 8, 2020
The cronjob tries to get key from `mastodon` secret instead of
`mastodon-postgresql` - so the cronjob fails with this error:

Error: couldn't find key postgresql-password in Secret [NS]/mastodon

Another solution is to save the postgres password in mastodon secret,
but that means that the password is placed in two places.

Postgresql use <fullname>-postgresql name as secret name.
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.

None yet

3 participants