Related T499
test.archiver: update the tests to follow changes in the archiver
Paths
| Differential D78 Authored by ardumont on Jul 20 2016, 5:03 PM.
Active Operations
Details
Summary Related T499 test.archiver: update the tests to follow changes in the archiver
Diff Detail
Event Timelineqcampos updated this object. Herald added a reviewer: Reviewers. · View Herald TranscriptJul 20 2016, 5:03 PM2016-07-20 17:03:36 (UTC+2) Comment Actions Other than the missed parameter to remove, this seems ok. As a note, that would not have shocked me if you have left the name storage instead of renaming it to storage.
ardumont edited edge metadata. This revision is now accepted and ready to land.Jul 20 2016, 6:01 PM2016-07-20 18:01:29 (UTC+2) This revision now requires review to proceed.Jul 20 2016, 6:10 PM2016-07-20 18:10:04 (UTC+2) Closed by commit rDSTOf500ca2a0612: archiver: change the master's storage to an object storage (authored by qcampos, committed by ardumont). · Explain WhyJul 20 2016, 6:59 PM2016-07-20 18:59:08 (UTC+2) This revision was automatically updated to reflect the committed changes. Comment Actions After some fights with phab, arcanist, etc... I finally successfully merged it directly. Thanks @qcampos ^^
Revision Contents
Diff 256 swh/storage/archiver/copier.py
swh/storage/archiver/director.py
swh/storage/archiver/tasks.py
swh/storage/archiver/worker.py
|
You forgot to remove the argument dbconn_storage here.