This will be used to compute an intrisic identifier for RawExtrinsicMetadata;
which can be used for deduplication and refering to it like any other sha1_git
instead of needed to use a tuple of its fields.
Details
- Reviewers
anlambert olasd - Group Reviewers
Reviewers - Maniphest Tasks
- T2703: Use intrinsic identifiers/hashes for RawExtrinsicMetadata objects
- Commits
- rDMODf6eab95253f1: identifiers: Add raw_extrinsic_metadata_identifier
Diff Detail
- Repository
- rDMOD Data model
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 18988 Build 29418: Phabricator diff pipeline on jenkins Jenkins console · Jenkins Build 29417: arc lint + arc unit
Event Timeline
Build is green
Patch application report for D4935 (id=17550)
Rebasing onto 9af451fd62...
First, rewinding head to replay your work on top of it... Applying: identifiers: Add raw_extrinsic_metadata_identifier
Changes applied before test
commit 1c195e7b099042dc4a6a7fcea57a95e91002e2b5 Author: Valentin Lorentz <vlorentz@softwareheritage.org> Date: Mon Jan 25 12:31:12 2021 +0100 identifiers: Add raw_extrinsic_metadata_identifier This will be used to compute an intrisic identifier for RawExtrinsicMetadata; which can be used for deduplication and refering to it like any other sha1_git instead of needed to use a tuple of its fields.
See https://jenkins.softwareheritage.org/job/DMOD/job/tests-on-diff/202/ for more details.
Build is green
Patch application report for D4935 (id=17552)
Rebasing onto 9af451fd62...
First, rewinding head to replay your work on top of it... Applying: identifiers: Add raw_extrinsic_metadata_identifier
Changes applied before test
commit b0b3e195b638734c94ebbcc8cab3b155f2137705 Author: Valentin Lorentz <vlorentz@softwareheritage.org> Date: Mon Jan 25 12:31:12 2021 +0100 identifiers: Add raw_extrinsic_metadata_identifier This will be used to compute an intrisic identifier for RawExtrinsicMetadata; which can be used for deduplication and refering to it like any other sha1_git instead of needed to use a tuple of its fields.
See https://jenkins.softwareheritage.org/job/DMOD/job/tests-on-diff/203/ for more details.
swh/model/identifiers.py | ||
---|---|---|
747 | I never remember which is the default encoding; can we be explicit here? |
for the records:
12:57 <vlorentz> olasd: if I have to add an argument to .encode(), then it makes lines 725 and 729 harder to read 12:57 <vlorentz> (because they go over 88 cols) 12:58 <+olasd> vlorentz: I guess the fact that the explicit .encodes() use ascii makes the default one more obvious 12:58 <+olasd> (i.e. don't sweat it)
Build is green
Patch application report for D4935 (id=17746)
Rebasing onto cad940dc8c...
Current branch diff-target is up to date.
Changes applied before test
commit 2a807789f22fdbf3838684b6b1ce8cf5a599b754 Author: Valentin Lorentz <vlorentz@softwareheritage.org> Date: Mon Jan 25 12:31:12 2021 +0100 identifiers: Add raw_extrinsic_metadata_identifier This will be used to compute an intrisic identifier for RawExtrinsicMetadata; which can be used for deduplication and refering to it like any other sha1_git instead of needed to use a tuple of its fields.
See https://jenkins.softwareheritage.org/job/DMOD/job/tests-on-diff/205/ for more details.
Build is green
Patch application report for D4935 (id=17866)
Rebasing onto 0c16581283...
Current branch diff-target is up to date.
Changes applied before test
commit 272468f3b5a96c8854a26efe333c32cba4504aff Author: Valentin Lorentz <vlorentz@softwareheritage.org> Date: Mon Jan 25 12:31:12 2021 +0100 identifiers: Add raw_extrinsic_metadata_identifier This will be used to compute an intrisic identifier for RawExtrinsicMetadata; which can be used for deduplication and refering to it like any other sha1_git instead of needed to use a tuple of its fields.
See https://jenkins.softwareheritage.org/job/DMOD/job/tests-on-diff/210/ for more details.
Build is green
Patch application report for D4935 (id=18552)
Could not rebase; Attempt merge onto 8e0119962b...
Updating 8e01199..d88a5e1 Fast-forward swh/model/cli.py | 92 +++++--- swh/model/hashutil.py | 9 +- swh/model/identifiers.py | 263 +++++++--------------- swh/model/tests/test_cli.py | 6 +- swh/model/tests/test_identifiers.py | 432 ++++++++++-------------------------- 5 files changed, 258 insertions(+), 544 deletions(-)
Changes applied before test
commit d88a5e13f2ffea5c0ebfad24e29ba41e86af20c0 Author: Valentin Lorentz <vlorentz@softwareheritage.org> Date: Mon Jan 25 12:31:12 2021 +0100 identifiers: Add raw_extrinsic_metadata_identifier This will be used to compute an intrisic identifier for RawExtrinsicMetadata; which can be used for deduplication and refering to it like any other sha1_git instead of needed to use a tuple of its fields. commit bf4ab4336f7b43d442988c47d3dd70bb82b595c5 Author: Valentin Lorentz <vlorentz@softwareheritage.org> Date: Wed Mar 3 10:44:48 2021 +0100 identifiers: Remove the deprecated SWHID class Other packages don't use it anymore. commit 1e924e84198a895003d6f649b8e3471cd93a7c7b Author: Valentin Lorentz <vlorentz@softwareheritage.org> Date: Wed Mar 3 10:44:27 2021 +0100 cli: stop using the deprecated SWHID class
See https://jenkins.softwareheritage.org/job/DMOD/job/tests-on-diff/273/ for more details.
Build is green
Patch application report for D4935 (id=18586)
Rebasing onto bf4ab4336f...
Current branch diff-target is up to date.
Changes applied before test
commit f6eab95253f13f28fe4d4652fc471e3e8a0b5565 Author: Valentin Lorentz <vlorentz@softwareheritage.org> Date: Mon Jan 25 12:31:12 2021 +0100 identifiers: Add raw_extrinsic_metadata_identifier This will be used to compute an intrisic identifier for RawExtrinsicMetadata; which can be used for deduplication and refering to it like any other sha1_git instead of needed to use a tuple of its fields.
See https://jenkins.softwareheritage.org/job/DMOD/job/tests-on-diff/277/ for more details.