Page MenuHomeSoftware Heritage
Differential D8640

Pubdev: Do not rely on intrinsic metadata
ClosedPublic
Actions

Authored by franckbret on Oct 7 2022, 12:01 PM.

Details

Reviewers
vlorentz
Group Reviewers
Reviewers
Commits
rDLDBASE4cb85e153e2e: Pubdev: Do not rely on intrinsic metadata
Summary

The loader get enough information from extrinsic metadata to build a release object, checking intrinsic metadata was more error prone than useful.

It should fix some Sentry reported errors.

Remove 'information' and adapt release message

Adapt loader specifications documentation

Related T4465, T4530, T4583

Diff Detail

Event Timeline

franckbret created this revision.Oct 7 2022, 12:01 PM
Herald added a reviewer: Reviewers. · View Herald TranscriptOct 7 2022, 12:01 PM
swh-public-ci added a comment.Oct 7 2022, 12:04 PM

Build is green

Patch application report for D8640 (id=31202)

Rebasing onto 8aa6dab72a...

Current branch diff-target is up to date.
Changes applied before test
commit deabeaf51feff0746fa00ecb2ba574b5fa02f1e6
Author: Franck Bret <franck.bret@octobus.net>
Date:   Fri Oct 7 11:51:35 2022 +0200

    Pubdev: Do not rely on intrinsic metadata
    
    The loader get enough information from extrinsic metadata to build a release object, checking intrinsic metadata was more error prone than useful.
    
    It should fix some Sentry reported errors.
    
    Remove 'information' and adapt release message
    
    Adapt loader specifications documentation
    
    Related T4465, T4530, T4583

commit 74289c868125a4d08743a1b2f00a4cd22410e1ad
Author: Franck Bret <franck.bret@octobus.net>
Date:   Wed Sep 28 16:23:45 2022 +0200

    Conda: Anaconda packages archive loader
    
    For each origin it takes advantage of 'artifacts' data send through
    'extra_loader_arguments' of the conda lister, providing versions,
    archive url, checksum, etc.
    Author and description are extracted from intrinsic metadata.
    
    Related T4579

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

Harbormaster completed remote builds in B32158: Diff 31202.Oct 7 2022, 12:04 PM
franckbret requested review of this revision.Oct 7 2022, 12:04 PM
franckbret updated this revision to Diff 31204.Oct 7 2022, 12:29 PM

Remove files from commit

swh-public-ci added a comment.Oct 7 2022, 12:33 PM

Build is green

Patch application report for D8640 (id=31204)

Could not rebase; Attempt merge onto 8aa6dab72a...

Updating 8aa6dab..498ab61
Fast-forward
 docs/package-loader-specifications.rst             |  13 +-
 setup.py                                           |   1 +
 swh/loader/package/conda/__init__.py               |  17 ++
 swh/loader/package/conda/loader.py                 | 168 +++++++++++++++
 swh/loader/package/conda/tasks.py                  |  14 ++
 swh/loader/package/conda/tests/__init__.py         |   0
 swh/loader/package/conda/tests/data/fake_conda.sh  | 231 +++++++++++++++++++++
 ...inux-64_lifetimes-0.11.1-py36h9f0ad1d_1.tar.bz2 | Bin 0 -> 1742 bytes
 ...inux-64_lifetimes-0.11.1-py36hc560c46_1.tar.bz2 | Bin 0 -> 1286 bytes
 swh/loader/package/conda/tests/test_conda.py       | 133 ++++++++++++
 swh/loader/package/conda/tests/test_tasks.py       |  24 +++
 swh/loader/package/pubdev/loader.py                |  67 ++----
 swh/loader/package/pubdev/tests/test_pubdev.py     |  21 +-
 13 files changed, 621 insertions(+), 68 deletions(-)
 create mode 100644 swh/loader/package/conda/__init__.py
 create mode 100644 swh/loader/package/conda/loader.py
 create mode 100644 swh/loader/package/conda/tasks.py
 create mode 100644 swh/loader/package/conda/tests/__init__.py
 create mode 100644 swh/loader/package/conda/tests/data/fake_conda.sh
 create mode 100644 swh/loader/package/conda/tests/data/https_conda.anaconda.org/conda-forge_linux-64_lifetimes-0.11.1-py36h9f0ad1d_1.tar.bz2
 create mode 100644 swh/loader/package/conda/tests/data/https_conda.anaconda.org/conda-forge_linux-64_lifetimes-0.11.1-py36hc560c46_1.tar.bz2
 create mode 100644 swh/loader/package/conda/tests/test_conda.py
 create mode 100644 swh/loader/package/conda/tests/test_tasks.py
Changes applied before test
commit 498ab615f4ec305390b58f93f7894311042f332e
Author: Franck Bret <franck.bret@octobus.net>
Date:   Fri Oct 7 11:51:35 2022 +0200

    Pubdev: Do not rely on intrinsic metadata
    
    The loader get enough information from extrinsic metadata to build a release object, checking intrinsic metadata was more error prone than useful.
    
    It should fix some Sentry reported errors.
    
    Remove 'information' and adapt release message
    
    Adapt loader specifications documentation
    
    Related T4465, T4530, T4583

commit 74289c868125a4d08743a1b2f00a4cd22410e1ad
Author: Franck Bret <franck.bret@octobus.net>
Date:   Wed Sep 28 16:23:45 2022 +0200

    Conda: Anaconda packages archive loader
    
    For each origin it takes advantage of 'artifacts' data send through
    'extra_loader_arguments' of the conda lister, providing versions,
    archive url, checksum, etc.
    Author and description are extracted from intrinsic metadata.
    
    Related T4579

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

Harbormaster completed remote builds in B32159: Diff 31204.Oct 7 2022, 12:33 PM
vlorentz added a subscriber: vlorentz.Oct 7 2022, 12:56 PM
vlorentz added inline comments.
swh/loader/package/pubdev/loader.py
115–117

what is the purpose of any(authors)? is it possible to have empty author names in the list?

franckbret added inline comments.Oct 7 2022, 4:31 PM
swh/loader/package/pubdev/loader.py
115–117

Can"t remember, maybe when doing docker I met list with empty string

vlorentz added inline comments.Oct 10 2022, 9:45 AM
swh/loader/package/pubdev/loader.py
115–117

then you shouldn't unconditionally take the first string if it may be empty.

(and please add a regression test)

anlambert added a subscriber: anlambert.Oct 10 2022, 1:26 PM
anlambert added inline comments.
swh/loader/package/pubdev/loader.py
106

As we are querying the pubdev Web API for getting package info, I think we should store the JSON data associated to a specific version as extrinsic metadata.
Nevertheless, this is out of scope for that diff and should be handled in a new one.

franckbret updated this revision to Diff 31238.Oct 11 2022, 7:23 AM
franckbret marked an inline comment as done.

Remove useless check condition when getttinh 'atuhor' data

swh-public-ci added a comment.Oct 11 2022, 7:27 AM

Build is green

Patch application report for D8640 (id=31238)

Could not rebase; Attempt merge onto 028b7c04b9...

Updating 028b7c0..350f632
Fast-forward
 docs/package-loader-specifications.rst             |  13 +-
 setup.py                                           |   1 +
 swh/loader/package/conda/__init__.py               |  17 ++
 swh/loader/package/conda/loader.py                 | 168 +++++++++++++++
 swh/loader/package/conda/tasks.py                  |  14 ++
 swh/loader/package/conda/tests/__init__.py         |   0
 swh/loader/package/conda/tests/data/fake_conda.sh  | 231 +++++++++++++++++++++
 ...inux-64_lifetimes-0.11.1-py36h9f0ad1d_1.tar.bz2 | Bin 0 -> 1742 bytes
 ...inux-64_lifetimes-0.11.1-py36hc560c46_1.tar.bz2 | Bin 0 -> 1286 bytes
 swh/loader/package/conda/tests/test_conda.py       | 133 ++++++++++++
 swh/loader/package/conda/tests/test_tasks.py       |  24 +++
 swh/loader/package/pubdev/loader.py                |  67 ++----
 swh/loader/package/pubdev/tests/test_pubdev.py     |  21 +-
 13 files changed, 621 insertions(+), 68 deletions(-)
 create mode 100644 swh/loader/package/conda/__init__.py
 create mode 100644 swh/loader/package/conda/loader.py
 create mode 100644 swh/loader/package/conda/tasks.py
 create mode 100644 swh/loader/package/conda/tests/__init__.py
 create mode 100644 swh/loader/package/conda/tests/data/fake_conda.sh
 create mode 100644 swh/loader/package/conda/tests/data/https_conda.anaconda.org/conda-forge_linux-64_lifetimes-0.11.1-py36h9f0ad1d_1.tar.bz2
 create mode 100644 swh/loader/package/conda/tests/data/https_conda.anaconda.org/conda-forge_linux-64_lifetimes-0.11.1-py36hc560c46_1.tar.bz2
 create mode 100644 swh/loader/package/conda/tests/test_conda.py
 create mode 100644 swh/loader/package/conda/tests/test_tasks.py
Changes applied before test
commit 350f632e412dc1434bd85e96033220e16cd9bd43
Author: Franck Bret <franck.bret@octobus.net>
Date:   Fri Oct 7 11:51:35 2022 +0200

    Pubdev: Do not rely on intrinsic metadata
    
    The loader get enough information from extrinsic metadata to build a release object, checking intrinsic metadata was more error prone than useful.
    
    It should fix some Sentry reported errors.
    
    Remove 'information' and adapt release message
    
    Adapt loader specifications documentation
    
    Related T4465, T4530, T4583

commit 4e067fd726a124a85d7ab24a2dc584932a82c151
Author: Franck Bret <franck.bret@octobus.net>
Date:   Wed Sep 28 16:23:45 2022 +0200

    Conda: Anaconda packages archive loader
    
    For each origin it takes advantage of 'artifacts' data send through
    'extra_loader_arguments' of the conda lister, providing versions,
    archive url, checksum, etc.
    Author and description are extracted from intrinsic metadata.
    
    Related T4579

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

Harbormaster completed remote builds in B32191: Diff 31238.Oct 11 2022, 7:27 AM
vlorentz added a comment.Oct 11 2022, 9:57 AM
In D8640#225200, @franckbret wrote:

Remove useless check condition when getttinh 'atuhor' data

You checked there is no empty string in the input data, right?

franckbret added inline comments.Oct 11 2022, 9:58 AM
swh/loader/package/pubdev/loader.py
115–117

I've runned a script to check the whole data set and did not found a case where authors is not a list or is a list with empty strings.

DEBUG:10/11/2022 07:05:37 AM:Found 34924 packages, 298689 versions, 82619 author, 11462 authors, 204608 empty author

So using Any here is useless for sure.

franckbret added a comment.Oct 11 2022, 9:58 AM
In D8640#225229, @vlorentz wrote:
In D8640#225200, @franckbret wrote:

Remove useless check condition when getttinh 'atuhor' data

You checked there is no empty string in the input data, right?

yes

vlorentz accepted this revision.Oct 11 2022, 10:25 AM

thanks

This revision is now accepted and ready to land.Oct 11 2022, 10:25 AM
franckbret marked an inline comment as done.Oct 11 2022, 10:56 AM
franckbret added inline comments.
swh/loader/package/pubdev/loader.py
106

yep sure, will add raw_extrinsic metadata in another patch

franckbret updated this revision to Diff 31241.Oct 11 2022, 10:57 AM
franckbret marked an inline comment as done.

rebase

swh-public-ci added a comment.Oct 11 2022, 11:01 AM

Build is green

Patch application report for D8640 (id=31241)

Could not rebase; Attempt merge onto 028b7c04b9...

Updating 028b7c0..350f632
Fast-forward
 docs/package-loader-specifications.rst             |  13 +-
 setup.py                                           |   1 +
 swh/loader/package/conda/__init__.py               |  17 ++
 swh/loader/package/conda/loader.py                 | 168 +++++++++++++++
 swh/loader/package/conda/tasks.py                  |  14 ++
 swh/loader/package/conda/tests/__init__.py         |   0
 swh/loader/package/conda/tests/data/fake_conda.sh  | 231 +++++++++++++++++++++
 ...inux-64_lifetimes-0.11.1-py36h9f0ad1d_1.tar.bz2 | Bin 0 -> 1742 bytes
 ...inux-64_lifetimes-0.11.1-py36hc560c46_1.tar.bz2 | Bin 0 -> 1286 bytes
 swh/loader/package/conda/tests/test_conda.py       | 133 ++++++++++++
 swh/loader/package/conda/tests/test_tasks.py       |  24 +++
 swh/loader/package/pubdev/loader.py                |  67 ++----
 swh/loader/package/pubdev/tests/test_pubdev.py     |  21 +-
 13 files changed, 621 insertions(+), 68 deletions(-)
 create mode 100644 swh/loader/package/conda/__init__.py
 create mode 100644 swh/loader/package/conda/loader.py
 create mode 100644 swh/loader/package/conda/tasks.py
 create mode 100644 swh/loader/package/conda/tests/__init__.py
 create mode 100644 swh/loader/package/conda/tests/data/fake_conda.sh
 create mode 100644 swh/loader/package/conda/tests/data/https_conda.anaconda.org/conda-forge_linux-64_lifetimes-0.11.1-py36h9f0ad1d_1.tar.bz2
 create mode 100644 swh/loader/package/conda/tests/data/https_conda.anaconda.org/conda-forge_linux-64_lifetimes-0.11.1-py36hc560c46_1.tar.bz2
 create mode 100644 swh/loader/package/conda/tests/test_conda.py
 create mode 100644 swh/loader/package/conda/tests/test_tasks.py
Changes applied before test
commit 350f632e412dc1434bd85e96033220e16cd9bd43
Author: Franck Bret <franck.bret@octobus.net>
Date:   Fri Oct 7 11:51:35 2022 +0200

    Pubdev: Do not rely on intrinsic metadata
    
    The loader get enough information from extrinsic metadata to build a release object, checking intrinsic metadata was more error prone than useful.
    
    It should fix some Sentry reported errors.
    
    Remove 'information' and adapt release message
    
    Adapt loader specifications documentation
    
    Related T4465, T4530, T4583

commit 4e067fd726a124a85d7ab24a2dc584932a82c151
Author: Franck Bret <franck.bret@octobus.net>
Date:   Wed Sep 28 16:23:45 2022 +0200

    Conda: Anaconda packages archive loader
    
    For each origin it takes advantage of 'artifacts' data send through
    'extra_loader_arguments' of the conda lister, providing versions,
    archive url, checksum, etc.
    Author and description are extracted from intrinsic metadata.
    
    Related T4579

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

Harbormaster completed remote builds in B32194: Diff 31241.Oct 11 2022, 11:01 AM
franckbret updated this revision to Diff 31243.Oct 11 2022, 12:20 PM

rebase

swh-public-ci added a comment.Oct 11 2022, 12:23 PM

Build is green

Patch application report for D8640 (id=31243)

Could not rebase; Attempt merge onto 028b7c04b9...

Updating 028b7c0..99cec3e
Fast-forward
 docs/package-loader-specifications.rst             |  13 +-
 setup.py                                           |   1 +
 swh/loader/package/conda/__init__.py               |  17 ++
 swh/loader/package/conda/loader.py                 | 168 +++++++++++++++
 swh/loader/package/conda/tasks.py                  |  14 ++
 swh/loader/package/conda/tests/__init__.py         |   0
 swh/loader/package/conda/tests/data/fake_conda.sh  | 231 +++++++++++++++++++++
 ...inux-64_lifetimes-0.11.1-py36h9f0ad1d_1.tar.bz2 | Bin 0 -> 1742 bytes
 ...inux-64_lifetimes-0.11.1-py36hc560c46_1.tar.bz2 | Bin 0 -> 1286 bytes
 swh/loader/package/conda/tests/test_conda.py       | 133 ++++++++++++
 swh/loader/package/conda/tests/test_tasks.py       |  24 +++
 swh/loader/package/pubdev/loader.py                |  67 ++----
 swh/loader/package/pubdev/tests/test_pubdev.py     |  21 +-
 13 files changed, 621 insertions(+), 68 deletions(-)
 create mode 100644 swh/loader/package/conda/__init__.py
 create mode 100644 swh/loader/package/conda/loader.py
 create mode 100644 swh/loader/package/conda/tasks.py
 create mode 100644 swh/loader/package/conda/tests/__init__.py
 create mode 100644 swh/loader/package/conda/tests/data/fake_conda.sh
 create mode 100644 swh/loader/package/conda/tests/data/https_conda.anaconda.org/conda-forge_linux-64_lifetimes-0.11.1-py36h9f0ad1d_1.tar.bz2
 create mode 100644 swh/loader/package/conda/tests/data/https_conda.anaconda.org/conda-forge_linux-64_lifetimes-0.11.1-py36hc560c46_1.tar.bz2
 create mode 100644 swh/loader/package/conda/tests/test_conda.py
 create mode 100644 swh/loader/package/conda/tests/test_tasks.py
Changes applied before test
commit 99cec3e03357a5bd75a2b2012299749540f5cc7f
Author: Franck Bret <franck.bret@octobus.net>
Date:   Fri Oct 7 11:51:35 2022 +0200

    Pubdev: Do not rely on intrinsic metadata
    
    The loader get enough information from extrinsic metadata to build a release object, checking intrinsic metadata was more error prone than useful.
    
    It should fix some Sentry reported errors.
    
    Remove 'information' and adapt release message
    
    Adapt loader specifications documentation
    
    Related T4465, T4530, T4583

commit 4e067fd726a124a85d7ab24a2dc584932a82c151
Author: Franck Bret <franck.bret@octobus.net>
Date:   Wed Sep 28 16:23:45 2022 +0200

    Conda: Anaconda packages archive loader
    
    For each origin it takes advantage of 'artifacts' data send through
    'extra_loader_arguments' of the conda lister, providing versions,
    archive url, checksum, etc.
    Author and description are extracted from intrinsic metadata.
    
    Related T4579

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

Harbormaster completed remote builds in B32195: Diff 31243.Oct 11 2022, 12:24 PM
franckbret updated this revision to Diff 31247.Oct 11 2022, 12:58 PM

rebase

This revision was landed with ongoing or failed builds.Oct 11 2022, 12:59 PM
Closed by commit rDLDBASE4cb85e153e2e: Pubdev: Do not rely on intrinsic metadata (authored by franckbret). · Explain Why
This revision was automatically updated to reflect the committed changes.
franckbret added a commit: rDLDBASE4cb85e153e2e: Pubdev: Do not rely on intrinsic metadata.
swh-public-ci added a comment.Oct 11 2022, 1:02 PM

Build is green

Patch application report for D8640 (id=31247)

Rebasing onto 028b7c04b9...

Current branch diff-target is up to date.
Changes applied before test
commit 4cb85e153e2ef7ed1fb51c16be324ea1568df30a
Author: Franck Bret <franck.bret@octobus.net>
Date:   Fri Oct 7 11:51:35 2022 +0200

    Pubdev: Do not rely on intrinsic metadata
    
    The loader get enough information from extrinsic metadata to build a release object, checking intrinsic metadata was more error prone than useful.
    
    It should fix some Sentry reported errors.
    
    Remove 'information' and adapt release message
    
    Adapt loader specifications documentation
    
    Related T4465, T4530, T4583

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

Harbormaster completed remote builds in B32199: Diff 31247.Oct 11 2022, 1:02 PM

Revision Contents
Changeset List

PathSizeCoverage (All)Coverage (Touched)
docs/
4 lines--
swh/
loader/
package/
pubdev/
67 lines100%
Loading...
tests/
21 lines100%
Loading...

Diff 31241

View Options

docs/package-loader-specifications.rst

Loading...
View Options

swh/loader/package/pubdev/loader.py

Loading...
View Options

swh/loader/package/pubdev/tests/test_pubdev.py

Loading...
About · Developer information · Legal