Differential D8519
Use generic HTTP retry policy by default and rename dedicated decorator Authored by anlambert on Sep 21 2022, 5:02 PM.
Details
Instead of retrying HTTP requests only for 429 status code by default, Rename throttling_retry decorator to http_retry to reflect this change.
Diff Detail
Event TimelineComment Actions Build is green Patch application report for D8519 (id=30680)Rebasing onto bd35d54398... Current branch diff-target is up to date. Changes applied before testcommit 6284d34b1725ce9c5e9f76359a2760da871edb7b
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date: Wed Sep 21 16:56:34 2022 +0200
Use generic HTTP retry policy by default and rename dedicated decorator
Instead of retrying HTTP requests only for 429 status code by default,
prefer to use the generic retry policy enabling to also retry for status
codes >= 500 but also on ConnectionError exceptions.
Rename throttling_retry decorator to http_retry to reflect this change.See https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/671/ for more details. Comment Actions Ok I'm not so sure in real life, 500 -> 500 -> 200 will happen that much often but let's see. Comment Actions Build is green Patch application report for D8519 (id=30739)Rebasing onto 9b3e565cf7... Current branch diff-target is up to date. Changes applied before testcommit 9c55acd286091acb6f6094e9fe1c95aca1fdeeec
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date: Wed Sep 21 16:56:34 2022 +0200
Use generic HTTP retry policy by default and rename dedicated decorator
Instead of retrying HTTP requests only for 429 status code by default,
prefer to use the generic retry policy enabling to also retry for status
codes >= 500 but also on ConnectionError exceptions.
Rename throttling_retry decorator to http_retry to reflect this change.See https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/684/ for more details. |