Page MenuHomeSoftware Heritage

Make Storage.origin_visit_update() add an OriginVisit model entity in the journal
ClosedPublic

Authored by douardda on Apr 7 2020, 4:45 PM.

Details

Summary

instead of a dict, to comply with next version of swh.journal (which will
require swh.model objects).

Diff Detail

Repository
rDSTO Storage manager
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

This revision is now accepted and ready to land.Apr 7 2020, 4:46 PM

Build has FAILED

Patch application report for D2971 (id=10563)

Could not rebase; Attempt merge onto 82b41bac01...

Updating 82b41ba..b6ebbec
Fast-forward
 setup.cfg              |  6 ++++++
 swh/storage/storage.py | 12 ++++++------
 2 files changed, 12 insertions(+), 6 deletions(-)
 create mode 100644 setup.cfg
Changes applied before test
commit b6ebbec89795cecea575f34d70a12206a581bf1c
Author: David Douard <david.douard@sdfa3.org>
Date:   Tue Apr 7 16:09:48 2020 +0200

    Make Storage.origin_visit_update() add an OriginVisit model entity in the journal
    
    instead of a dict, to comply with next version of swh.journal (which will
    require swh.model objects).

commit b7ad557128a504f9719acc08266d56fe40baf0a3
Author: David Douard <david.douard@sdfa3.org>
Date:   Tue Apr 7 16:09:05 2020 +0200

    Make swh/storage/storage.py flake8 compliant

commit 5b711ecb275c032540663dd08b7c591ad409b12b
Author: David Douard <david.douard@sdfa3.org>
Date:   Tue Apr 7 16:06:25 2020 +0200

    Add a setup.cfg file to configure flake8 for black compatibility

Link to build: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/59/
See console output for more information: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/59/console

This revision was landed with ongoing or failed builds.Apr 8 2020, 12:08 PM
This revision was automatically updated to reflect the committed changes.

Build has FAILED

Patch application report for D2971 (id=10580)

Rebasing onto c51139ef96...

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

Link to build: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/66/
See console output for more information: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/66/console