Page MenuHomeSoftware Heritage

Fixes and improvements for T401
ClosedPublic

Authored by qcampos on May 13 2016, 4:50 PM.

Diff Detail

Repository
rDSTO Storage manager
Branch
T401
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 100
Build 140: Software Heritage Python tests
Build 139: arc lint + arc unit

Event Timeline

qcampos retitled this revision from to Fixes and improvements for T401.
qcampos updated this object.
qcampos edited the test plan for this revision. (Show Details)
qcampos edited edge metadata.
  • Improve test coverage of the archiver

Depends on D27

I just tried to use a "Depends on" keyword in diff summary in order to create a second diff D28 relative to D27. Didn't work as expected, it's just do the same than arc diff --update D27.
However, the last commit that improve the test coverage can be reviewed separately but depends strongly on the other fixes and refactoring.

ardumont added inline comments.
swh/storage/archiver/director.py
43–54

typo: must contain

48

typo: to copy all

swh/storage/archiver/worker.py
26

s/where is the content location/(db connection url, content location)/

28

For the sake of coherence with the rest of your diff, you forgot to adapt the docstring on config argument.

38

arguments.

203–219

If one copier fails significantly badly (e.g. by throwing something), the remaining copier won't even be triggered.
We may want to trigger the other ones even though one or more fail.
This could dealt with in the run_copier method.

207

You might want to add in the docstring the fact that if the copier fails to run (returns False), we consider the content not being copied (because no update takes place).

210

...archive_url) of the destination.

211

List of contents

swh/storage/tests/test_archiver.py
128

'enough '
Ctrl-t just before the h in emacs will transpose the blank character and the t ^^

qcampos marked 10 inline comments as done.
qcampos edited edge metadata.

Correct some typo and an uncatched exception.

This comment was removed by qcampos.
qcampos edited edge metadata.

More typo corrections

This revision is now accepted and ready to land.May 17 2016, 12:39 PM
qcampos edited edge metadata.

Last typo I hope

This revision now requires review to proceed.May 17 2016, 12:39 PM
This revision is now accepted and ready to land.May 17 2016, 12:40 PM
This revision was automatically updated to reflect the committed changes.
This revision was automatically updated to reflect the committed changes.
This revision was automatically updated to reflect the committed changes.
zack changed the visibility from "All Users" to "Public (No Login Required)".