Page MenuHomeSoftware Heritage

Use the new API for origin_metadata.
ClosedPublic

Authored by vlorentz on Apr 29 2020, 4:07 PM.

Details

Summary

Mutually dependent on D2988.

Test Plan

tox

Diff Detail

Repository
rDLDBASE Generic VCS/Package Loader
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Build has FAILED

Patch application report for D3091 (id=10992)

Rebasing onto fca7e4783d...

Current branch diff-target is up to date.
Changes applied before test
commit 58edc3618d801e807d6fa5c7950d7bbc33982c85
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Wed Apr 29 16:07:05 2020 +0200

    Use the new API for origin_metadata.

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

ardumont added a subscriber: ardumont.

sounds good.

swh/loader/package/deposit/loader.py
134

This is the internal deposit_read api the loader use (the one i was talking about earlier) [1]

[1] https://forge.softwareheritage.org/source/swh-deposit/browse/master/swh/deposit/api/private/deposit_read.py$111-127

151

i guess those are respectively the authority and fetcher's id.

This revision is now accepted and ready to land.Apr 29 2020, 7:56 PM
swh/loader/package/deposit/loader.py
151

yes

This needs a rebase on latest loader deposit adaptations.
Then land and tag.

Otherwise, since D2988 landed and got released, we will no longer have a loader
deposit working.

Build is green

Patch application report for D3091 (id=11248)

Rebasing onto 8bd38f9bb4...

Current branch diff-target is up to date.
Changes applied before test
commit 1e9b83cdf895c9a82c4cd8be5aacc76762f4232b
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Wed Apr 29 16:07:05 2020 +0200

    Use the new API for origin_metadata.

See https://jenkins.softwareheritage.org/job/DLDBASE/job/tests-on-diff/35/ for more details.

This revision was automatically updated to reflect the committed changes.