Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1520)

Unified Diff: appengine/findit/handlers/config.py

Issue 2526963002: [Findit] Implement retry within swarming_util.py when making server calls (Closed)
Patch Set: Self-review Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: appengine/findit/handlers/config.py
diff --git a/appengine/findit/handlers/config.py b/appengine/findit/handlers/config.py
index 8e713e24e14c73ec4c6c5a03fbca7e55dead141a..a003227e24a11ff4a14337869c069b38646744d5 100644
--- a/appengine/findit/handlers/config.py
+++ b/appengine/findit/handlers/config.py
@@ -167,7 +167,11 @@ def _ValidateSwarmingSettings(settings):
isinstance(settings.get('iterations_to_rerun'), int) and
isinstance(
settings.get('get_swarming_task_id_timeout_seconds'), int) and
- isinstance(settings.get('get_swarming_task_id_wait_seconds'), int))
+ isinstance(settings.get('get_swarming_task_id_wait_seconds'), int) and
+ isinstance(settings.get('server_retry_timeout'), int) and
+ isinstance(settings.get(
+ 'maximum_server_contact_retry_interval_seconds'), int) and
+ isinstance(settings.get('should_retry_server'), bool))
def _ValidateDownloadBuildDataSettings(settings):

Powered by Google App Engine
This is Rietveld 408576698