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

Unified Diff: scripts/master/factory/swarm_factory.py

Issue 10035003: Split up Each Swarm Test into Two Steps (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Renaming broken test :( Created 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « scripts/master/factory/swarm_commands.py ('k') | scripts/slave/get_swarm_results.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/master/factory/swarm_factory.py
diff --git a/scripts/master/factory/swarm_factory.py b/scripts/master/factory/swarm_factory.py
index 38faf9cf6f7d910da774389a91d924de67b93468..6a6f40b188aee6df8f7d886c37565ffa05e75baf 100644
--- a/scripts/master/factory/swarm_factory.py
+++ b/scripts/master/factory/swarm_factory.py
@@ -26,15 +26,21 @@ class SwarmFactory(chromium_factory.ChromiumFactory):
self._target_platform)
gclient_env = factory_properties.get("gclient_env")
+ swarm_server = factory_properties.get('swarm_server', 'http://localhost')
gyp_defines = gclient_env['GYP_DEFINES']
if 'tests_run=hashtable' in gyp_defines:
- for test in tests:
- swarm_command_obj.AddSwarmTestStep(self._target_platform,
- factory_properties.get('swarm_server', 'localhost'),
- factory_properties.get('swarm_port', '9001'),
+ # Send of all the test requests as a single step.
+ swarm_inputs = [os.path.join('src', 'out', target, test + '.results')
+ for test in tests]
+ swarm_command_obj.AddTriggerSwarmTestStep(self._target_platform,
+ swarm_server,
factory_properties.get('min_swarm_shards', '3'),
factory_properties.get('max_swarm_shards', '3'),
- os.path.join('src', 'out', target, test + '.results'),
- test)
+ swarm_inputs,
+ tests)
+
+ # Each test has its output returned as its own step.
+ for test in tests:
+ swarm_command_obj.AddGetSwarmTestStep(swarm_server, test)
return factory
« no previous file with comments | « scripts/master/factory/swarm_commands.py ('k') | scripts/slave/get_swarm_results.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698