Page MenuHomeSoftware Heritage

api servers: Unify the server starting point with production
ClosedPublic

Authored by ardumont on Thu, Oct 10, 10:47 AM.

Details

Summary

Related D2109

(Also add a bit of deposit definition cleanup)

Test Plan
  1. doco up
  2. check api services impacted (deposit, webapp, objstorage, storage, indexer-storage)

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

ardumont created this revision.Thu, Oct 10, 10:47 AM
ardumont retitled this revision from api server: Unify the server starting point with production to api servers: Unify the server starting point with production.Thu, Oct 10, 11:05 AM
ardumont updated this revision to Diff 7075.Thu, Oct 10, 11:17 AM
  • deposit: Remove indirections in deposit definitions
vlorentz requested changes to this revision.Thu, Oct 10, 11:20 AM
vlorentz added inline comments.
docker-compose.yml
246–249 ↗(On Diff #7075)

why not in a .env file?

This revision now requires changes to proceed.Thu, Oct 10, 11:20 AM
ardumont added inline comments.Thu, Oct 10, 11:22 AM
docker-compose.yml
246–249 ↗(On Diff #7075)

It's:

  • clearer to read here without indirection
  • used only once
  • deposit is kinda the same as swh-web same (django apps) which does this as well
ardumont edited the summary of this revision. (Show Details)Thu, Oct 10, 11:23 AM
ardumont edited the test plan for this revision. (Show Details)
vlorentz accepted this revision.Thu, Oct 10, 11:23 AM
This revision is now accepted and ready to land.Thu, Oct 10, 11:23 AM