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.

Sign up for GitHub

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

Jump to bottom

Database migration fails #54

Closed
joaodss2 opened this issue Sep 1, 2024 · 3 comments · Fixed by #55
Closed

Database migration fails #54

joaodss2 opened this issue Sep 1, 2024 · 3 comments · Fixed by #55

Comments

@joaodss2
Copy link

joaodss2 commented Sep 1, 2024 •
edited
Loading

Postgress extraScript needs to run manually so that wger-app can create database on postgres

Temporary solution:
run on postgres pod with postgress user

ALTER USER wger WITH SUPERUSER;

change wger to user under Values.postgres.userDatabase.name
@bbkz
Copy link
Contributor

bbkz commented Sep 13, 2024

Thank's for the report. Did i understood it right, the extraScript was not run in your case?

@joaodss2
Copy link
Author

it did run but the command fails on postgres, thus not giving admin rights to the user account, which then fails the migration due to insufficient rights.
I was able to fix the issue running the command locally

@bbkz
Copy link
Contributor

bbkz commented Sep 13, 2024

Ok, thank you for clarifying. I'll check that.

bbkz added a commit to bbkz/wger-helm-charts that referenced this issue Sep 14, 2024
@bbkz
fixes wger-project#54
334ab99
@bbkz bbkz mentioned this issue Sep 17, 2024
Merged
@bbkz bbkz closed this as completed in #55 Sep 17, 2024
bbkz added a commit that referenced this issue Sep 17, 2024
@bbkz
Merge pull request #55 from bbkz/dev
e06c70a 
Release 0.2.1

* fixes #54 Database migration fails
* fix celery redis password
* update development setup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Release 0.2.1 bbkz/wger-helm-charts
2 participants
@bbkz @joaodss2