Page MenuHomeSoftware Heritage

Add loader workers services
ClosedPublic

Authored by douardda on Jan 9 2019, 5:29 PM.

Details

Summary

Also

  • task_queues configuration is not needed any more
  • Fix loader config file name

We should really consider being more consistant here:

  • loaders: ~/.config/swh/loader/<name>.yml
  • listers: ~/.config/swh/lister-<name>.yml

Require D888, D900 and D901 to work properly.

Diff Detail

Repository
rCDFD Dockerfiles for developers
Branch
loaders
Lint
Lint Skipped
Unit
Unit Tests Skipped
Build Status
Buildable 3344
Build 4311: arc lint + arc unit

Event Timeline

vlorentz added a subscriber: vlorentz.
vlorentz added inline comments.
dockerfiles/swh-loaders-worker/Dockerfile
9–10

indent

dockerfiles/swh-loaders-worker/entrypoint.sh
8

will

scheduler.env
5

Did you mean to commit this?

This revision is now accepted and ready to land.Jan 9 2019, 5:52 PM
ardumont added a subscriber: ardumont.

\m/

We should really consider being more consistant here...

Quite!

consistant

also, probably a typo, still, it's consist`e`nt ;)

douardda added inline comments.
scheduler.env
5

Defining the env var is needed here (according the entrypoint.sh file) and I don't mind running these dockerized env stuff in debug mode, but I'm fine with upgrading the log level to INFO here.

Fix typos and raise log level to INFO

This revision was automatically updated to reflect the committed changes.