Page MenuHomeSoftware Heritage
Differential D2368

Add throttle_scope swh_api_origin_search.
ClosedPublic
Actions

Authored by vlorentz on Nov 27 2019, 4:27 PM.

Details

Reviewers
anlambert
Group Reviewers
Reviewers
Commits
rDWAPPSe3212e06d566: Add throttle_scope swh_api_origin_search.
Summary

Needed by D2364. Added in prod config by D2366

Diff Detail

Repository
rDWAPPS Web applications
Branch
throttle-scope-swh_api_origin_search
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 9409
Build 13793: Cypress tests for swh-web diffsJenkins
Build 13792: tox-on-jenkinsJenkins
Build 13791: arc lint + arc unit

Event Timeline

vlorentz created this revision.Nov 27 2019, 4:27 PM
Herald added a reviewer: Reviewers. · View Herald TranscriptNov 27 2019, 4:27 PM
vlorentz edited the summary of this revision. (Show Details)Nov 27 2019, 4:27 PM
anlambert accepted this revision.Nov 27 2019, 4:30 PM
anlambert added a subscriber: anlambert.

It will be better if this is aligned with the production config (10/m).

This revision is now accepted and ready to land.Nov 27 2019, 4:30 PM
swh-public-ci added a comment.Nov 27 2019, 4:31 PM

Build has FAILED

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

Harbormaster failed remote builds in B9409: Diff 8339!Nov 27 2019, 4:31 PM
vlorentz updated this revision to Diff 8340.Nov 27 2019, 4:31 PM

align with prod config.

vlorentz updated this revision to Diff 8342.Nov 27 2019, 4:32 PM

rebase

This revision was landed with ongoing or failed builds.Nov 27 2019, 4:32 PM
Closed by commit rDWAPPSe3212e06d566: Add throttle_scope swh_api_origin_search. (authored by vlorentz). · Explain Why
This revision was automatically updated to reflect the committed changes.
vlorentz added a commit: rDWAPPSe3212e06d566: Add throttle_scope swh_api_origin_search..
swh-public-ci added a comment.Nov 27 2019, 4:34 PM

Build has FAILED

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

Harbormaster failed remote builds in B9410: Diff 8340!Nov 27 2019, 4:34 PM
Harbormaster failed remote builds in B9412: Diff 8342!
swh-public-ci added a comment.Nov 27 2019, 4:36 PM

Build has FAILED

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

anlambert added a comment.Nov 27 2019, 4:39 PM

Oops, I forgot that tests config must also been updated in swh/web/settings/tests.py.

The code overriding the config in it should be improved to avoid some config entries duplication.

swh-public-ci added a comment.Nov 27 2019, 4:40 PM

Build is green
See https://jenkins.softwareheritage.org/job/DWAPPS/job/cypress-diff/406/ for more details.

swh-public-ci added a comment.Nov 27 2019, 4:40 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/cypress-diff/407/
See console output for more information: https://jenkins.softwareheritage.org/job/DWAPPS/job/cypress-diff/407/console

swh-public-ci added a comment.Nov 27 2019, 4:42 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/cypress-diff/409/
See console output for more information: https://jenkins.softwareheritage.org/job/DWAPPS/job/cypress-diff/409/console

Revision Contents
Changeset List

PathSize
swh/
web/
api/
views/
3 lines
6 lines

Diff 8339

View Options

swh/web/api/views/origin.py

Loading...
View Options

swh/web/config.py

Loading...
About · Developer information · Legal