They were already deprecated.
This will allow simplifying code in swh.storage.
Paths
| Differential D1969 Authored by vlorentz on Sep 6 2019, 4:44 PM.
Details
Summary They were already deprecated. This will allow simplifying code in swh.storage.
Diff Detail
Event TimelineHerald added a reviewer: Reviewers. · View Herald TranscriptSep 6 2019, 4:44 PM2019-09-06 16:44:11 (UTC+2) Comment Actions Build is green Comment Actions Build is green Harbormaster completed remote builds in B7718: Diff 6637.Sep 6 2019, 4:54 PM2019-09-06 16:54:53 (UTC+2) This revision is now accepted and ready to land.Sep 9 2019, 10:54 AM2019-09-09 10:54:22 (UTC+2) Comment Actions For your information, I extracted all requests to swh-web in production containing /origin/ in their urls for the last 1 year (using the following Python script P522): 141892 requests were found. As the associated elasticsearch index does not handle regexp query properly, I filtered the urls based on origin ids using grep ($ cat swh-web-origin-requests | grep -E /origin/[0-9]+) All requests using origin ids for the last 1 year (aka today minus 1 year range) can be found here: P523 (for a total of 852 requests) Almost all found requests consists querying an internal browse endpoint that has been removed since (rDWAPPS4762da3d332114f97bbebd86941d43a236929537). Otherwise, only 18 requests to Web API endpoints using origin ids have been issued. So removing the endpoints using origin ids seems pretty safe, considering they are not really used. This revision was landed with ongoing or failed builds.Sep 10 2019, 11:57 AM2019-09-10 11:57:17 (UTC+2) Closed by commit rDWAPPSb74602b9a54a: Remove all API endpoints using an origin id. (authored by vlorentz). · Explain Why This revision was automatically updated to reflect the committed changes. Harbormaster failed remote builds in B7743: Diff 6660!Sep 10 2019, 11:57 AM2019-09-10 11:57:57 (UTC+2) Comment Actions Build has FAILED Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/tox/721/ Comment Actions Build has FAILED Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/cypress-diff/325/
Revision Contents
Diff 6661 swh/web/api/views/origin.py
swh/web/api/views/revision.py
|