swh-web: Ensure to use a new empty database in dev mode
ClosedPublic
Actions

Authored by anlambert on Jun 15 2021, 11:47 AM.

Details

Reviewers

ardumont

Group Reviewers

Reviewers

Commits

rDENV9de769abbf4b: docker/conf/swh-web: Ensure to use a new empty database in dev mode

Summary

When overriding swh-web sources, default development database will be
used but it corresponds to the local swh-web development database on
the host sytem using sqlite3 backend.

So ensure to use a newly created sqlite3 database located in the
swh-web service container instead to not persist data across docker
environment sessions.

Diff Detail

Repository

rDENV Development environment

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 21997
Build 34221: arc lint + arc unit

Event Timeline

anlambert created this revision.Jun 15 2021, 11:47 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptJun 15 2021, 11:47 AM

Harbormaster completed remote builds in B21997: Diff 21019.Jun 15 2021, 11:47 AM

anlambert requested review of this revision.Jun 15 2021, 11:47 AM

An alternative would be to use a postgres db in docker instead.

But fine, let's see how it goes.

This revision is now accepted and ready to land.Jun 15 2021, 11:59 AM

In D5869#149887, @ardumont wrote:

An alternative would be to use a postgres db in docker instead.

But fine, let's see how it goes.

swh-web django development settings can only use sqlite3 dbs at the moment, meaning some adaptations are required in the settings if we want to use any db backend,

Closed by commit rDENV9de769abbf4b: docker/conf/swh-web: Ensure to use a new empty database in dev mode (authored by anlambert). · Explain WhyJun 15 2021, 12:14 PM

This revision was automatically updated to reflect the committed changes.

anlambert added a commit: rDENV9de769abbf4b: docker/conf/swh-web: Ensure to use a new empty database in dev mode.

swh-web django development settings can only use sqlite3 dbs at the moment, meaning some adaptations are required in the settings if we want to use any db backend,

indeed, i had forgotten about that part.