Page MenuHomeSoftware Heritage
Differential D1870

Remove person_get endpoint.
ClosedPublic
Actions

Authored by vlorentz on Aug 19 2019, 4:47 PM.

Details

Reviewers
olasd
Group Reviewers
Reviewers
Commits
rDSTOC57f60cdabf24: Remove person_get endpoint.
rDSTO57f60cdabf24: Remove person_get endpoint.
Summary

swh-web is about to stop using it, and was the only user of
this endpoint.

Should not be merged before D1869 is released.

Diff Detail

Repository
rDSTO Storage manager
Branch
rm-person_get
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 7336
Build 10417: tox-on-jenkinsJenkins
Build 10416: arc lint + arc unit

Event Timeline

vlorentz created this revision.Aug 19 2019, 4:47 PM
Herald added a reviewer: Reviewers. · View Herald TranscriptAug 19 2019, 4:47 PM
vlorentz edited the summary of this revision. (Show Details)Aug 19 2019, 4:48 PM
swh-public-ci added a comment.Aug 19 2019, 4:52 PM

Build is green
See https://jenkins.softwareheritage.org/job/DSTO/job/tox/579/ for more details.

Harbormaster completed remote builds in B7311: Diff 6275.Aug 19 2019, 4:52 PM
olasd accepted this revision.Aug 19 2019, 6:03 PM
This revision is now accepted and ready to land.Aug 19 2019, 6:03 PM
vlorentz added a child revision: D1871: Stop leaking person ids..Aug 20 2019, 2:09 PM
vlorentz updated this revision to Diff 6299.Aug 20 2019, 2:57 PM

rebase

This revision was landed with ongoing or failed builds.Aug 20 2019, 2:57 PM
Closed by commit rDSTO57f60cdabf24: Remove person_get endpoint. (authored by vlorentz). · Explain Why
This revision was automatically updated to reflect the committed changes.
Harbormaster failed remote builds in B7336: Diff 6299!Aug 20 2019, 2:57 PM
swh-public-ci added a comment.Aug 20 2019, 2:57 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DSTO/job/tox/585/
See console output for more information: https://jenkins.softwareheritage.org/job/DSTO/job/tox/585/console

vlorentz added a commit: rDSTOC57f60cdabf24: Remove person_get endpoint..Oct 30 2019, 5:21 PM

Revision Contents
Changeset List

PathSize
swh/
storage/
api/
3 lines
6 lines
16 lines
18 lines
14 lines
tests/
49 lines

Diff 6299

View Options

swh/storage/api/client.py

Loading...
View Options

swh/storage/api/server.py

Loading...
View Options

swh/storage/db.py

Loading...
View Options

swh/storage/in_memory.py

Loading...
View Options

swh/storage/storage.py

Loading...
View Options

swh/storage/tests/test_storage.py

Loading...
About · Developer information · Legal