Buildbot json for swarming Mac 10.10 Perf on perf waterfall

tools/perf/generate_perf_json.py

Index: tools/perf/generate_perf_json.py
diff --git a/tools/perf/generate_perf_json.py b/tools/perf/generate_perf_json.py
index db045ac6fa87ae2019e8d412321a8161b651cba4..df6c41f559996c725eb5f554d9375f30f3ab0b18 100755
--- a/tools/perf/generate_perf_json.py
+++ b/tools/perf/generate_perf_json.py
@@ -318,7 +318,15 @@ def get_waterfall_config():
     'mac', num_host_shards=5)
   waterfall = add_tester(
     waterfall, 'Mac 10.10 Perf', 'chromium-rel-mac10',
-    'mac', num_host_shards=5)
+    'mac',
+    swarming=[
+      {
+       'os': 'Mac-10.10',
+       'device_ids': [
+           'build158-m1', 'build159-m1', 'build160-m1',
+           'build161-m1', 'build162-m1']
+      }
+    ])
   waterfall = add_tester(
     waterfall, 'Mac Retina Perf',
     'chromium-rel-mac-retina', 'mac', num_host_shards=5)
@@ -447,12 +455,14 @@ def generate_telemetry_tests(tester_config, benchmarks, benchmark_sharding_map):
       device_id = dimension['device_ids'][device_affinity]
       # Id is unique within the swarming pool so it is the only needed
       # identifier for the bot to run the test on
-      swarming_dimensions.append({
+      complete_dimension = {
         'id': device_id,
-        'gpu': dimension['gpu'],
         'os': dimension['os'],
         'pool': 'Chrome-perf',
-      })
+      }
+      if dimension.get('gpu', None):

[ghost stip (do not use), 2016/11/09 21:28:34]

nit: the none is implicit, you can just do dimension.get('gpu')

[dtu, 2016/11/09 23:43:11]

I think the most Pythonic way is
if 'gpu' in dimension:

[eyaich1, 2016/11/10 00:22:47]

Done.

[eyaich1, 2016/11/10 00:22:47]

Done.

+        complete_dimension['gpu'] = dimension['gpu']
+      swarming_dimensions.append(complete_dimension)
 
     test = generate_telemetry_test(
       swarming_dimensions, benchmark.Name(), browser_name)