They are not mandatory anymore.
Details
Details
- Reviewers
ardumont - Group Reviewers
Reviewers - Maniphest Tasks
- T2806: Add tag in the deposit protocol to allow client to specify an origin
- Commits
- rDDEP5989af38c9e6: Don't generate slugs on the client side
Diff Detail
Diff Detail
- Repository
- rDDEP Push deposit
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 17571 Build 27163: Phabricator diff pipeline on jenkins Jenkins console · Jenkins Build 27162: arc lint + arc unit
Event Timeline
Comment Actions
Build is green
Patch application report for D4607 (id=16350)
Could not rebase; Attempt merge onto ad6041d312...
Updating ad6041d3..f863ea93 Fast-forward docs/endpoints/collection.rst | 2 -- docs/specs/spec-loading.rst | 3 +- docs/specs/swh.xsd | 35 ++++++++++++------- swh/deposit/api/collection.py | 12 ++----- swh/deposit/api/common.py | 39 +++++++++------------- swh/deposit/cli/client.py | 16 ++++----- swh/deposit/client.py | 19 +++++++---- swh/deposit/errors.py | 15 --------- .../0021_deposit_origin_url_20201124_1438.py | 28 ++++++++++++++++ swh/deposit/models.py | 9 +++-- swh/deposit/tests/api/test_collection_post_atom.py | 19 ++++++++--- .../tests/api/test_collection_post_binary.py | 18 +++++++--- .../tests/api/test_collection_post_multipart.py | 18 +++++++--- .../api/test_deposit_private_read_metadata.py | 13 +++----- swh/deposit/tests/cli/test_client.py | 8 ++--- 15 files changed, 146 insertions(+), 108 deletions(-) create mode 100644 swh/deposit/migrations/0021_deposit_origin_url_20201124_1438.py
Changes applied before test
commit f863ea93a22b3132749261d479f9d2a8117d52bd
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Thu Nov 26 14:45:48 2020 +0100
Don't generate slugs on the client side
They are not mandatory anymore.
commit fa3a3591e6932e16c13890f45b1bd26296954e74
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Wed Nov 25 17:49:13 2020 +0100
Make the Slug header optional.
To follow the SWORD specification.
commit 45b596f198df963edb2d6f5353763704b16fd05d
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Wed Nov 25 15:50:47 2020 +0100
Defer origin_url computation until the deposit completes.
In a future commit, the origin_url computation will only be a fallback
in case it's not provided in any of the Atom entries; so we have
to compute this fallback after all Atom entries are given, ie. when
the deposit completes.
commit 1a2da5e1805f7d65cdd6c22b6180ad6fb00d623a
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Tue Nov 24 15:52:36 2020 +0100
add column 'origin_url' on Deposit.
For now, it's only computed based on the external_id, but clients
will provide it in future versions.
commit 2e4a524a2480dbd3baa6f59390528fd694ea7390
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Tue Nov 24 12:20:19 2020 +0100
swh.xsd: Use the https://www.softwareheritage.org/schema/2018/deposit namespace
commit 88b102aec44185b97b06135d862821bb4c138acc
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Tue Nov 24 12:11:31 2020 +0100
swh.xsd: Improve readability
This also moves the inner element definitions of <swh:reference> to
root elements, so they can be reused by new definitions in the future.See https://jenkins.softwareheritage.org/job/DDEP/job/tests-on-diff/445/ for more details.
Comment Actions
Build is green
Patch application report for D4607 (id=16379)
Could not rebase; Attempt merge onto ad6041d312...
Updating ad6041d3..5989af38 Fast-forward docs/endpoints/collection.rst | 2 -- docs/specs/spec-loading.rst | 3 +- docs/specs/swh.xsd | 35 ++++++++++++------- swh/deposit/api/collection.py | 12 ++----- swh/deposit/api/common.py | 39 +++++++++------------- swh/deposit/cli/client.py | 16 ++++----- swh/deposit/client.py | 19 +++++++---- swh/deposit/errors.py | 15 --------- .../0021_deposit_origin_url_20201124_1438.py | 28 ++++++++++++++++ swh/deposit/models.py | 9 +++-- swh/deposit/tests/api/test_collection_post_atom.py | 19 ++++++++--- .../tests/api/test_collection_post_binary.py | 18 +++++++--- .../tests/api/test_collection_post_multipart.py | 18 +++++++--- .../api/test_deposit_private_read_metadata.py | 13 +++----- swh/deposit/tests/cli/test_client.py | 8 ++--- 15 files changed, 146 insertions(+), 108 deletions(-) create mode 100644 swh/deposit/migrations/0021_deposit_origin_url_20201124_1438.py
Changes applied before test
commit 5989af38c9e6b3d5e32e20bd125aa149d6668e8a
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Thu Nov 26 14:45:48 2020 +0100
Don't generate slugs on the client side
They are not mandatory anymore.
commit fc5a5ee0b1c67c90c3eaf5a924a6c46eb6fb0f7f
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Wed Nov 25 17:49:13 2020 +0100
Make the Slug header optional.
To follow the SWORD specification.
commit e7316d6d8a527eb4a96eb2e9d46daa0045c6b851
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Wed Nov 25 15:50:47 2020 +0100
Defer origin_url computation until the deposit completes.
In a future commit, the origin_url computation will only be a fallback
in case it's not provided in any of the Atom entries; so we have
to compute this fallback after all Atom entries are given, ie. when
the deposit completes.
commit a21961284e3b88a3c21fc0b67cae793351331e7f
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Tue Nov 24 15:52:36 2020 +0100
add column 'origin_url' on Deposit.
For now, it's only computed based on the external_id, but clients
will provide it in future versions.
commit 2e4a524a2480dbd3baa6f59390528fd694ea7390
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Tue Nov 24 12:20:19 2020 +0100
swh.xsd: Use the https://www.softwareheritage.org/schema/2018/deposit namespace
commit 88b102aec44185b97b06135d862821bb4c138acc
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Tue Nov 24 12:11:31 2020 +0100
swh.xsd: Improve readability
This also moves the inner element definitions of <swh:reference> to
root elements, so they can be reused by new definitions in the future.See https://jenkins.softwareheritage.org/job/DDEP/job/tests-on-diff/456/ for more details.