Skip to content

Postgresql - #296

Draft
ericgaspar wants to merge 26 commits into
testingfrom
postgresql
Draft

Postgresql#296
ericgaspar wants to merge 26 commits into
testingfrom
postgresql

Conversation

@ericgaspar

@ericgaspar ericgaspar commented Aug 17, 2025

Copy link
Copy Markdown
Member

@ericgaspar
ericgaspar marked this pull request as draft August 17, 2025 09:12
@ericgaspar

Copy link
Copy Markdown
Member Author

!testme

@yunohost-bot

Copy link
Copy Markdown
Contributor

Remember: a failing test is just a misunderstood spicy feature!
Test Badge

Comment thread scripts/upgrade Outdated
#=================================================

# This migration should be done before the upgrade
if psql -lqt | cut -d \| -f 1 | grep -qw "$db_name"; then

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess upgrade is stuck here?
I'd simply run if ynh_app_upgrading_from_version_before because it only has to run once

@ericgaspar

Copy link
Copy Markdown
Member Author

!testme

@yunohost-bot

Copy link
Copy Markdown
Contributor

Alrighty!
Test Badge

@Thovi98

Thovi98 commented Nov 6, 2025

Copy link
Copy Markdown
Member

I tried migrating on a test instance, it failed:
https://paste.yunohost.org/raw/alutanoqos

@ewilly

ewilly commented Nov 8, 2025

Copy link
Copy Markdown
Member

I tried migrating on a test instance, it failed: https://paste.yunohost.org/raw/alutanoqos

Perhaps due to point 3 in https://github.com/dani-garcia/vaultwarden/wiki/Using-the-PostgreSQL-Backend#migrating-from-sqlite-to-postgresql : start it, so diesel can run migrations and set up the schema properly which should be done before migrating datas

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.

4 participants