Add CT isolate file and CT script that will be run on swarming slaves

chrome/ct_top1k.isolate.tmpl

Index: chrome/ct_top1k.isolate.tmpl
diff --git a/chrome/ct_top1k.isolate.tmpl b/chrome/ct_top1k.isolate.tmpl
new file mode 100644
index 0000000000000000000000000000000000000000..9cdb0a2879d07b80dfebc1c0f431e9398110f628
--- /dev/null
+++ b/chrome/ct_top1k.isolate.tmpl
@@ -0,0 +1,44 @@
+# Copyright (c) 2015 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+{
+  'includes': [
+    # TODO(rmistry): Try out the below once things are running on the
+    #                FYI waterfall.
+    # '../tools/telemetry/telemetry.isolate',
+    # '../tools/perf/perf.isolate',
+    '../tools/perf/chrome_telemetry_build/telemetry_binary_manager.isolate',
+  ],
+  'conditions': [
+    ['OS=="android" or OS=="linux" or OS=="mac" or OS=="win"', {
+      'variables': {
+        'files': [
+          '../build/android/pylib/',
+          '../build/android/devil/',
+          '../testing/variations/',
+          '../third_party/catapult/',
+          '../tools/telemetry/',
+          '../tools/perf/',
+          '../tools/variations/',
+          '../testing/test_env.py',
+          '../out/Release/',
+
+          '../content/test/ct/run_ct_top1k.py',
+          '../content/test/ct/path_util.py',
+          '../content/test/ct/slave[[SLAVE_NUM]]/page_sets/',
+          '../content/test/ct/run_chromium_perf_swarming',
+        ],
+        'command': [
+          'python',
+          '../content/test/ct/run_ct_top1k.py',
+          '--slave_num=[[SLAVE_NUM]]',

[M-A Ruel, 2015/11/16 20:22:02]

You should use native isolate variable support.

Then you don't need to make a template. It's used via the --path-variable that
is also usable for arguments.

[rmistry, 2015/11/17 13:00:56]

That sounds good. I modified things and tried to pass in:
--path-variable MASTER chromium.perf.fyi
--path-variable SLAVE_NUM 1

but it fails with:

  File
"/repos/chromium-tools/build/slave/fake_slave/build/swarming.client/isolate.py",
line 153, in <genexpr>
    for k, v in path_variables.iteritems())
  File
"/repos/chromium-tools/build/slave/fake_slave/build/swarming.client/isolate.py",
line 130, in _normalize_path_variable
    raise ExecutionError('%s=%s is not a directory' % (key, normalized))
ExecutionError:
MASTER=/repos/chromium-tools/build/slave/fake_slave/build/src/chrome/chromium.perf.fyi
is not a directory


Looks like it expects a directory and always fails if it is not a dir. Is there
another way to pass in args?

[rmistry, 2015/11/17 15:57:46]

Ah, it works with --extra-variable. Done.

+          '--benchmark=[[BENCHMARK]]',
+          '--master=[[MASTER]]',
+          '--builder=[[BUILDER]]',
+          '--git_hash=[[GIT_HASH]]',
+        ],
+      },
+    }],
+  ]
+}