Page MenuHomeSoftware Heritage
Differential D5391

swh-web: Migrate from sqlite to postgresql
ClosedPublic
Actions

Authored by ardumont on Mar 31 2021, 3:28 PM.

Details

Reviewers
anlambert
Group Reviewers
Reviewers
Maniphest Tasks
T2945: Migrate swh-web production database from SQLite to PostgreSQL
Commits
rDENV482fa7d1068e: swh-web: Migrate from sqlite to postgresql
Summary

Providing the swh.web.settings.production is adapted accordingly [1]

This allows the webapp to use a postgresql backend.

Related to T2945

[1] D5392

Test Plan

doco up -d

webapp starts and uses the postgres backend.

Diff Detail

Repository
rDENV Development environment
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 20372
Build 31625: arc lint + arc unit

Event Timeline

ardumont requested review of this revision.Mar 31 2021, 3:28 PM
ardumont created this revision.
Harbormaster completed remote builds in B20372: Diff 19304.Mar 31 2021, 3:28 PM
ardumont edited the summary of this revision. (Show Details)Mar 31 2021, 3:32 PM
ardumont mentioned this in D5392: swh-web: Migrate from sqlite to postgresql.Mar 31 2021, 4:31 PM
ardumont updated this revision to Diff 19307.Mar 31 2021, 4:55 PM
  • Rename db key to production_db key
  • Adapt missed web*.yml configuration according to the main change.
Harbormaster completed remote builds in B20375: Diff 19307.Mar 31 2021, 4:55 PM
anlambert added inline comments.Mar 31 2021, 5:00 PM
docker/docker-compose.yml
229–230

Those files are missing from the diff I guess.

ardumont added inline comments.Mar 31 2021, 5:12 PM
docker/docker-compose.yml
229–230

oh, that may be well possible, i have too much ungitted noise (that i use once in a while)...

ardumont updated this revision to Diff 19308.Mar 31 2021, 5:14 PM

Add missing env files... (nice catch @anlambert)

Harbormaster completed remote builds in B20376: Diff 19308.Mar 31 2021, 5:14 PM
ardumont mentioned this in T2945: Migrate swh-web production database from SQLite to PostgreSQL.Apr 1 2021, 4:42 PM
anlambert accepted this revision.Apr 1 2021, 5:45 PM

LGTM

docker/env/01-web-db.env
2 ↗(On Diff #19308)

nitpick: missing new line at end of file

This revision is now accepted and ready to land.Apr 1 2021, 5:45 PM
Closed by commit rDENV482fa7d1068e: swh-web: Migrate from sqlite to postgresql (authored by ardumont). · Explain WhyApr 1 2021, 6:48 PM
This revision was automatically updated to reflect the committed changes.
ardumont added a commit: rDENV482fa7d1068e: swh-web: Migrate from sqlite to postgresql.
ardumont added inline comments.Apr 1 2021, 6:51 PM
docker/env/01-web-db.env
2 ↗(On Diff #19308)

arf, missed it, sorry

Revision Contents
Changeset List

PathSize
docker/
conf/
8 lines
7 lines
services/
swh-web/
5 lines

Diff 19304

View Options

docker/conf/web.yml

Loading...
View Options

docker/docker-compose.yml

Loading...
View Options

docker/services/swh-web/entrypoint.sh

Loading...
About · Developer information · Legal