Run telemetry_gpu_unittests via isolate on "Linux Tests" and trybot.

infra/scripts/runtest_wrapper.py

Index: infra/scripts/runtest_wrapper.py
diff --git a/infra/scripts/runtest_wrapper.py b/infra/scripts/runtest_wrapper.py
index fe9835f3bf401773b545420dd8f05374044d2bc1..aea8ec99bb8bedf2b8a175325bba430635386266 100755
--- a/infra/scripts/runtest_wrapper.py
+++ b/infra/scripts/runtest_wrapper.py
@@ -22,8 +22,12 @@ def main(argv):
   # TODO(phajdan.jr): Remove after cleaning up build repo side.
   parser.add_argument(
       '--path-build', help='Path to the build repo')
-  parser.add_argument('args', nargs='*', help='Arguments to pass to runtest.py')
+  parser.add_argument('args', nargs=argparse.REMAINDER,
+                      help='Arguments to pass to runtest.py')
   args = parser.parse_args(argv)
+  runtest_args = args.args
+  if runtest_args[0] == '--':

[Paweł Hajdan Jr., 2015/09/24 14:29:22]

Wait, why is this logic needed?

I wouldn't mind too much if this was some code specific to your test, but this
wrapper would live at least for some time (and I have further cleanup plans
related to it).

I might be able to suggest something here or even fix things myself - for now
however, I'd just like to understand what you're trying to do.

[nednguyen, 2015/09/24 15:54:54]

To use the isolate_script_test, we need to add the extra command args "--
--isolated-script-test-output" to the arguments list passed to
runtest_wrapper.py.

With the original logic, nargs='*' will skip all the "--" in the command line
arguments. (see the log in
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

Therefore I need to modify parser.add_argument('args', nargs='*'..) to
parser.add_argument('args', nargs=argparse.REMAINDER..).

However, the argument we pass to runtest_wrapper is: 
"--path-build /b/build -- --target Release --xvfb...", which means using
argparse.REMAINDER keep the "--" right before "--isolated-script-test-output",
but also the one right before "--target".

Hence I need to strip out the starting "--" here.

+    runtest_args.pop(0)
 
   env = copy.copy(os.environ)
   # Reset PYTHONPATH to make sure we're not accidentally using
@@ -42,7 +46,7 @@ def main(argv):
       sys.executable,
       os.path.join(SRC_DIR, 'infra', 'scripts', 'legacy',
                    'scripts', 'slave', 'runtest.py')
-  ] + args.args, env=env)
+  ] + runtest_args, env=env)
 
 
 if __name__ == '__main__':