Differential D1713

Increase rate-limit for swh_api_origin_visit_latest.
ClosedPublic
Actions

Authored by vlorentz on Jul 10 2019, 2:28 PM.

Tags

None

Subscribers

Details

Reviewers

anlambert
olasd

Group Reviewers

Commits

rSPSITE28004ffb4eaa: Increase rate-limit for swh_api_origin_visit_latest.

Summary

This endpoint is being introduced to replace a more data intensive
'internal' endpoint that wasn't rate-limited (_origin_latest_snapshot).

Related diffs: D1709 D1710 D1712

Diff Detail

Repository

rSPSITE puppet-swh-site

Branch

swh_api_origin_visit_latest

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 6773
Build 9475: arc lint + arc unit

Event Timeline

vlorentz created this revision.Jul 10 2019, 2:28 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptJul 10 2019, 2:28 PM

Harbormaster completed remote builds in B6773: Diff 5751.Jul 10 2019, 2:28 PM

vlorentz edited the summary of this revision. (Show Details)Jul 10 2019, 2:29 PM

vlorentz edited the summary of this revision. (Show Details)

vlorentz mentioned this in D1712: Add a specific throttling scope for api_origin_visit_latest..

anlambert accepted this revision.Jul 10 2019, 2:36 PM

anlambert added a subscriber: anlambert.

anlambert added inline comments.

data/defaults.yaml
1792	You should increase that value to at least 700/m I think. Starting the third page of results from the browse search interface (easily reachable in less than a minute), the visit statuses will not be correctly displayed otherwise.

This revision is now accepted and ready to land.Jul 10 2019, 2:36 PM

Increase to 700/m.

Harbormaster completed remote builds in B6774: Diff 5752.Jul 10 2019, 2:42 PM

vlorentz marked an inline comment as done.Jul 10 2019, 2:42 PM

olasd accepted this revision.Jul 10 2019, 2:45 PM

Closed by commit rSPSITE28004ffb4eaa: Increase rate-limit for swh_api_origin_visit_latest. (authored by vlorentz). · Explain WhyJul 10 2019, 2:46 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

data/

6 lines

Diff 5751

data/defaults.yaml

Loading...