Page MenuHomeSoftware Heritage

swh-web: enforce the database file to be created in either $HOME or /tmp
ClosedPublic

Authored by douardda on Apr 10 2019, 12:23 PM.

Details

Summary

but NOT in the swh-web python package.

Needed to be able to run swh-web installed from source in a docker
in which the code runs as swh user.

Diff Detail

Repository
rCDFD Dockerfiles for developers
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

douardda created this revision.Apr 10 2019, 12:23 PM

Looks sensible to me but we probably need a second opinion from @anlambert

I would rather adapt swh-web configuration in order to specify the database file to use.

Duplicating Django developement settings seems wrong to me as we will have to synchronize
it with every changes in swh.web.setting.development.

@douardda , any objection on this ?

anlambert accepted this revision.Apr 10 2019, 2:36 PM

Accepting this, I will update swh-web configuration afterwards.

This revision is now accepted and ready to land.Apr 10 2019, 2:36 PM
This revision was automatically updated to reflect the committed changes.