docker-dev: Use loader-package implems from swh.loader.core modules
ClosedPublic
Actions

Authored by ardumont on Nov 19 2019, 2:33 PM.

Details

Reviewers

douardda

Group Reviewers

Reviewers

Commits

rDENVcaa78d56528c: loaders: Use new loader-package from the swh.loader.core modules
rCDFDcaa78d56528c: loaders: Use new loader-package from the swh.loader.core modules

Summary

This also removes the potential [1] breaking of the different services (referencing the old task names without the module installed)

[1] I did not see it in the earlier diff (D2304).

I also did not yet tested it entirely (in-progress, that should not prevent from merging as it stabilizes a bit more already).
Now swh-loader image starts.

Related T1389

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.Nov 19 2019, 2:33 PM

Harbormaster completed remote builds in B9130: Diff 7938.Nov 19 2019, 2:33 PM

ardumont retitled this revision from docker-dev: Use loader-package implems from swh.loader.core modules to [wip] docker-dev: Use loader-package implems from swh.loader.core modules.Nov 19 2019, 2:38 PM

ardumont edited the summary of this revision. (Show Details)

ardumont added inline comments.Nov 19 2019, 2:42 PM

Dockerfile
34	It was implicitely installed as a dependency of other dvcs loaders. Better be explicit about it now that it's a direct dependency for package loaders.

ardumont mentioned this in D2306: .mrconfig: Stop referencing old package loader implementations.Nov 19 2019, 2:48 PM

ardumont mentioned this in D2307: req-swh*: Remove old package loader implementations references.Nov 19 2019, 2:52 PM

Remove some more old references which prevented the loader service to start

Harbormaster completed remote builds in B9133: Diff 7941.Nov 19 2019, 2:56 PM

ardumont edited the summary of this revision. (Show Details)Nov 19 2019, 2:58 PM

ardumont added a project: Docker environment.

douardda accepted this revision.Nov 19 2019, 3:03 PM

This revision is now accepted and ready to land.Nov 19 2019, 3:03 PM

ardumont retitled this revision from [wip] docker-dev: Use loader-package implems from swh.loader.core modules to docker-dev: Use loader-package implems from swh.loader.core modules.Nov 19 2019, 3:05 PM

ardumont edited the summary of this revision. (Show Details)

Closed by commit rCDFDcaa78d56528c: loaders: Use new loader-package from the swh.loader.core modules (authored by ardumont). · Explain WhyNov 19 2019, 3:05 PM

This revision was automatically updated to reflect the committed changes.

ardumont added a commit: rCDFDcaa78d56528c: loaders: Use new loader-package from the swh.loader.core modules.

ardumont mentioned this in rDSCHc973ec029d20: req-swh*: Remove old package loader backend names.Nov 19 2019, 4:28 PM

ardumont mentioned this in rDDOC70de3057fa48: req-swh*: Remove old package loader implementations references.Nov 19 2019, 4:58 PM

ardumont added a commit: rDENVcaa78d56528c: loaders: Use new loader-package from the swh.loader.core modules.Nov 26 2019, 1:35 PM