Details
Details
- Reviewers
ardumont - Group Reviewers
Reviewers - Maniphest Tasks
- T2400: Ingest current and historical Ubuntu releases
- Commits
- rDLS4ff3e44643f3: debian: Update extra_loader_arguments dict produced ListedOrigin models
Diff Detail
Diff Detail
- Repository
- rDLS Listers
- Branch
- debian-lister-update-listed-origin-fields
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 25342 Build 39613: Phabricator diff pipeline on jenkins Jenkins console · Jenkins Build 39612: arc lint + arc unit
Event Timeline
Comment Actions
Build is green
Patch application report for D6738 (id=24469)
Could not rebase; Attempt merge onto 5f567b3c34...
Updating 5f567b3..4ff3e44 Fast-forward swh/lister/debian/lister.py | 8 ++++++-- swh/lister/debian/tests/test_lister.py | 17 +++++++++++++---- 2 files changed, 19 insertions(+), 6 deletions(-)
Changes applied before test
commit 4ff3e44643f3f549e6dcf054b048fa92094fc5da
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date: Thu Dec 2 20:07:37 2021 +0100
debian: Update extra_loader_arguments dict produced ListedOrigin models
Remove no longer used date parameter in extra_loader_arguments.
Related to T2400
commit 46425917c25e4d58cff6ab8870eaddc13180bf92
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date: Thu Dec 2 17:19:08 2021 +0100
debian: Add missing file URIs in lister output
For a given package, the debian lister generates a dictionary mapping
distribution and version to a list of files to be processed by the
debian loader.
For each file to process, the debian loader expects to find an URI
in order to download it and then use its content to ingest package
source code into the archive.
However, it turns out these URIs were not computed by the lister
in its current implementation making any debian loading task fail
due to these missing info.
So add the computation of these URIS and ensure they will be provided
in the debian loader input parameters.
Related to T2400See https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/426/ for more details.