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

Side by Side Diff: scripts/slave/recipe_modules/v8/test_api.py

Issue 1906353002: Revert of V8: Show build environment in failure logs (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Created 4 years, 8 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 | Annotate | Revision Log
« no previous file with comments | « scripts/slave/recipe_modules/v8/api.py ('k') | scripts/slave/recipes/v8.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 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 # Exposes the builder and recipe configurations to GenTests in recipes. 5 # Exposes the builder and recipe configurations to GenTests in recipes.
6 6
7 import re 7 import re
8 8
9 from recipe_engine import recipe_test_api 9 from recipe_engine import recipe_test_api
10 from . import builders 10 from . import builders
(...skipping 350 matching lines...) Expand 10 before | Expand all | Expand 10 after
361 parent_buildername=parent_buildername, 361 parent_buildername=parent_buildername,
362 revision='20123', 362 revision='20123',
363 **kwargs 363 **kwargs
364 ) + 364 ) +
365 self.m.platform( 365 self.m.platform(
366 bot_config['testing']['platform'], 366 bot_config['testing']['platform'],
367 v8_config_kwargs.get('TARGET_BITS', 64), 367 v8_config_kwargs.get('TARGET_BITS', 64),
368 ) 368 )
369 ) 369 )
370 if parent_buildername: 370 if parent_buildername:
371 test += self.m.properties( 371 test += self.m.properties(parent_got_revision='54321')
372 parent_got_revision='54321',
373 build_environment={
374 'useful': 'envvars', 'from': 'the', 'parent': 'bot'},
375 )
376 if bot_config.get('enable_swarming'): 372 if bot_config.get('enable_swarming'):
377 # Assume each tester is triggered with the required hashes for all 373 # Assume each tester is triggered with the required hashes for all
378 # tests. 374 # tests.
379 test += self.m.properties( 375 test += self.m.properties(
380 parent_got_swarming_client_revision='[dummy swarming client hash]', 376 parent_got_swarming_client_revision='[dummy swarming client hash]',
381 swarm_hashes=self._make_dummy_swarm_hashes(bot_config), 377 swarm_hashes=self._make_dummy_swarm_hashes(bot_config),
382 ) 378 )
383 379
384 if mastername.startswith('tryserver'): 380 if mastername.startswith('tryserver'):
385 test += self.m.properties( 381 test += self.m.properties(
386 category='cq', 382 category='cq',
387 master='tryserver.v8', 383 master='tryserver.v8',
388 patch_project='v8', 384 patch_project='v8',
389 patch_storage='rietveld', 385 patch_storage='rietveld',
390 reason='CQ', 386 reason='CQ',
391 revision='12345', 387 revision='12345',
392 try_job_key='1234', 388 try_job_key='1234',
393 ) 389 )
394 390
395 return test 391 return test
396 392
397 def fail(self, step_name, variant='default'): 393 def fail(self, step_name, variant='default'):
398 return self.override_step_data( 394 return self.override_step_data(
399 step_name, self.failures_example(variant=variant)) 395 step_name, self.failures_example(variant=variant))
OLDNEW
« no previous file with comments | « scripts/slave/recipe_modules/v8/api.py ('k') | scripts/slave/recipes/v8.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698