Page MenuHomeSoftware Heritage
Differential D1709

Add API endpoint api_origin_visit_latest.
ClosedPublic
Actions

Authored by vlorentz on Jul 10 2019, 1:33 PM.

Details

Reviewers
anlambert
Group Reviewers
Reviewers
Commits
rDWAPPS2648d40a4cda: Add API endpoint api_origin_visit_latest.
Summary

It will replace the 'internal' endpoint _origin_latest_snapshot,
and won't require fetching the whole snapshot just to know
if there is a visit with a snapshot.

Depends on D1708.

Diff Detail

Repository
rDWAPPS Web applications
Branch
api_origin_visit_latest
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 6768
Build 9468: tox-on-jenkinsJenkins
Build 9467: arc lint + arc unit

Event Timeline

vlorentz created this revision.Jul 10 2019, 1:33 PM
Herald added a reviewer: Reviewers. · View Herald TranscriptJul 10 2019, 1:33 PM
vlorentz updated this revision to Diff 5740.Jul 10 2019, 1:34 PM

fix commit

vlorentz added a child revision: D1710: Use api_origin_visit_latest instead of _origin_latest_snapshot in origin search..Jul 10 2019, 1:37 PM
swh-public-ci added a comment.Jul 10 2019, 1:37 PM

Build has FAILED

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

Harbormaster failed remote builds in B6761: Diff 5739!Jul 10 2019, 1:37 PM
swh-public-ci added a comment.Jul 10 2019, 1:41 PM

Build has FAILED

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

Harbormaster failed remote builds in B6762: Diff 5740!Jul 10 2019, 1:41 PM
anlambert accepted this revision.Jul 10 2019, 1:50 PM
anlambert added a subscriber: anlambert.
anlambert added inline comments.
swh/web/common/service.py
862

Correct return parameters formatting using Napoleon style is:

dict: The origin_visit concerned
This revision is now accepted and ready to land.Jul 10 2019, 1:50 PM
vlorentz updated this revision to Diff 5743.Jul 10 2019, 2:01 PM

Fix compatibility with the current version of the storage.

vlorentz updated this revision to Diff 5746.Jul 10 2019, 2:04 PM

Fix docstring

vlorentz marked an inline comment as done.Jul 10 2019, 2:04 PM
Harbormaster completed remote builds in B6765: Diff 5743.Jul 10 2019, 2:04 PM
swh-public-ci added a comment.Jul 10 2019, 2:04 PM

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

This revision was landed with ongoing or failed builds.Jul 10 2019, 2:11 PM
Closed by commit rDWAPPS2648d40a4cda: Add API endpoint api_origin_visit_latest. (authored by vlorentz). · Explain Why
This revision was automatically updated to reflect the committed changes.
swh-public-ci added a comment.Jul 10 2019, 2:12 PM

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

Harbormaster completed remote builds in B6768: Diff 5746.Jul 10 2019, 2:12 PM
vlorentz mentioned this in D1712: Add a specific throttling scope for api_origin_visit_latest..Jul 10 2019, 2:29 PM
vlorentz mentioned this in D1713: Increase rate-limit for swh_api_origin_visit_latest..

Revision Contents
Changeset List

PathSize
swh/
web/
api/
views/
42 lines
common/
19 lines
tests/
api/
views/
73 lines

Diff 5746

View Options

swh/web/api/views/origin.py

Loading...
View Options

swh/web/common/service.py

Loading...
View Options

swh/web/tests/api/views/test_origin.py

Loading...
About · Developer information · Legal