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

Side by Side Diff: appengine/findit/waterfall/test/wf_testcase.py

Issue 2258373002: [Findit] Move configurable parameters in try_job_util.py to config (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Created 4 years, 4 months 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 unified diff | Download patch
OLDNEW
1 # Copyright 2016 The Chromium Authors. All rights reserved. 1 # Copyright 2016 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 from google.appengine.api import users 5 from google.appengine.api import users
6 6
7 import copy 7 import copy
8 8
9 from common.findit_testcase import FinditTestCase 9 from common.findit_testcase import FinditTestCase
10 from model.wf_config import FinditConfig 10 from model.wf_config import FinditConfig
(...skipping 57 matching lines...) Expand 10 before | Expand all | Expand 10 after
68 'mastername': 'tryserver.master', 68 'mastername': 'tryserver.master',
69 'buildername': 'tryserver.builder', 69 'buildername': 'tryserver.builder',
70 } 70 }
71 } 71 }
72 } 72 }
73 73
74 74
75 _DEFAULT_TRY_JOB_SETTINGS = { 75 _DEFAULT_TRY_JOB_SETTINGS = {
76 'server_query_interval_seconds': 60, 76 'server_query_interval_seconds': 60,
77 'job_timeout_hours': 5, 77 'job_timeout_hours': 5,
78 'allowed_response_error_times': 5 78 'allowed_response_error_times': 5,
79 'max_seconds_look_back_for_group': 86400
79 } 80 }
80 81
81 82
82 _DEFAULT_SWARMING_SETTINGS = { 83 _DEFAULT_SWARMING_SETTINGS = {
83 'server_host': 'chromium-swarm.appspot.com', 84 'server_host': 'chromium-swarm.appspot.com',
84 'default_request_priority': 150, 85 'default_request_priority': 150,
85 'request_expiration_hours': 20, 86 'request_expiration_hours': 20,
86 'server_query_interval_seconds': 60, 87 'server_query_interval_seconds': 60,
87 'task_timeout_hours': 23, 88 'task_timeout_hours': 23,
88 'isolated_server': 'https://isolateserver.appspot.com', 89 'isolated_server': 'https://isolateserver.appspot.com',
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 if config_property and override_data: 130 if config_property and override_data:
130 config_data = copy.deepcopy(DEFAULT_CONFIG_DATA) 131 config_data = copy.deepcopy(DEFAULT_CONFIG_DATA)
131 config_data[config_property].update(override_data) 132 config_data[config_property].update(override_data)
132 133
133 FinditConfig.Get().Update(users.User(email='admin@chromium.org'), True, 134 FinditConfig.Get().Update(users.User(email='admin@chromium.org'), True,
134 **config_data) 135 **config_data)
135 136
136 def setUp(self): 137 def setUp(self):
137 super(WaterfallTestCase, self).setUp() 138 super(WaterfallTestCase, self).setUp()
138 self.UpdateUnitTestConfigSettings() 139 self.UpdateUnitTestConfigSettings()
OLDNEW
« no previous file with comments | « appengine/findit/waterfall/test/waterfall_config_test.py ('k') | appengine/findit/waterfall/try_job_util.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698