Page MenuHomeSoftware Heritage
Differential D7124

Allow loader-bzr to run within docker
ClosedPublic
Actions

Authored by ardumont on Feb 8 2022, 5:10 PM.

Details

Reviewers
Alphare
Group Reviewers
Reviewers
Maniphest Tasks
T3915: Deploy Bzr loader to the staging environment
Commits
rDENVdd5e80612fd1: Allow loader-bzr to run within docker
Summary

Related to T3915

Test Plan

docker-compose up with docker-compose.override.yml (because no pypi uploaded module yet, next step)

Diff Detail

Repository
rDENV Development environment
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ardumont requested review of this revision.Feb 8 2022, 5:10 PM
ardumont created this revision.
Harbormaster completed remote builds in B26679: Diff 25836.Feb 8 2022, 5:10 PM
Alphare accepted this revision.Feb 8 2022, 5:12 PM
This revision is now accepted and ready to land.Feb 8 2022, 5:12 PM
ardumont updated this revision to Diff 25837.Feb 8 2022, 5:15 PM

Update task queue for the save code now as well

Alphare added a comment.Feb 8 2022, 5:16 PM

I don't understand why the tasks code is appearing here since I've sent it earlier today

ardumont mentioned this in T3915: Deploy Bzr loader to the staging environment.Feb 8 2022, 5:18 PM
swh-public-ci added a comment.Feb 8 2022, 5:19 PM

Build is green

Patch application report for D7124 (id=25837)

Rebasing onto 078fb50296...

First, rewinding head to replay your work on top of it...
Fast-forwarded diff-target to base-revision-28-D7124.
Changes applied before test



See https://jenkins.softwareheritage.org/job/DLDBZR/job/tests-on-diff/28/ for more details.
Harbormaster completed remote builds in B26680: Diff 25837.Feb 8 2022, 5:19 PM
ardumont updated this revision to Diff 25838.Feb 8 2022, 5:21 PM
Update diff from the right repo
Harbormaster completed remote builds in B26681: Diff 25838.Feb 8 2022, 5:21 PM
ardumont added a comment.Feb 8 2022, 5:21 PM
I don't understand why the tasks code is appearing here since I've sent it earlier today



Because i updated the diff from the loader-bzr repository instead of the swh-environment one.

Fixed!
ardumont updated this revision to Diff 25847.Feb 9 2022, 11:10 AM
Rebase
Harbormaster completed remote builds in B26689: Diff 25847.Feb 9 2022, 11:11 AM
Closed by commit rDENVdd5e80612fd1: Allow loader-bzr to run within docker (authored by ardumont).  ·  Explain WhyFeb 9 2022, 7:02 PM
This revision was automatically updated to reflect the committed changes.
ardumont added a commit: rDENVdd5e80612fd1: Allow loader-bzr to run within docker.
Revision Contents
Changeset List
PathSize
docker/
1 line
conf/
2 lines
Diff 25878
View Options
docker/Dockerfile
Loading...
View Options
docker/conf/loader.yml
Loading...
About · Developer information · Legal