Add --output option to Swarming client and pass it through to the bot

appengine/swarming/handlers_endpoints_test.py

 #!/usr/bin/env python
 # coding=utf-8
 # Copyright 2015 The LUCI Authors. All rights reserved.
 # Use of this source code is governed under the Apache License, Version 2.0
 # that can be found in the LICENSE file.
 
 import base64
 import datetime
 import json
 import logging
@@ skipping 213 matching lines @@
                 ],
                 server='https://chrome-infra-packages.appspot.com'),
             command=['rm', '-rf', '/'],
             dimensions=[
               swarming_rpcs.StringPair(key='pool', value='default'),
             ],
             env=[
               swarming_rpcs.StringPair(key='PATH', value='/'),
             ],
             execution_timeout_secs=30,
-            io_timeout_secs=30),
+            io_timeout_secs=30,
+            outputs=['foo','path/to/dir']),
         tags=['foo:bar'],
         user='joe@localhost',
         pubsub_topic='projects/abc/topics/def',
         pubsub_auth_token='secret that must not be shown',
         pubsub_userdata='userdata',
         service_account_token='bot')
     expected = {
       u'request': {
         u'authenticated': u'user:user@example.com',
         u'created_ts': str_now,
@@ skipping 17 matching lines @@
           u'dimensions': [
             {u'key': u'pool', u'value': u'default'},
           ],
           u'env': [
             {u'key': u'PATH', u'value': u'/'},
           ],
           u'execution_timeout_secs': u'30',
           u'grace_period_secs': u'30',
           u'idempotent': False,
           u'io_timeout_secs': u'30',
+          u'outputs': [u'foo', u'path/to/dir'],
         },
         u'pubsub_topic': u'projects/abc/topics/def',
         u'pubsub_userdata': u'userdata',
         u'service_account': u'bot',
         u'tags': [
           u'foo:bar',
           u'pool:default',
           u'priority:200',
           u'service_account:bot',
           u'user:joe@localhost',
@@ skipping 38 matching lines @@
                       path='bin',
                       version='git_revision:deadbeef'),
                 ],
                 server='https://chrome-infra-packages.appspot.com'),
             dimensions=[
               swarming_rpcs.StringPair(key='os', value='Amiga'),
               swarming_rpcs.StringPair(key='pool', value='default'),
             ],
             execution_timeout_secs=3600,
             idempotent=True,
-            io_timeout_secs=1200),
+            io_timeout_secs=1200,
+            outputs=['foo', 'path/to/foobar']),
         tags=['foo:bar'],
         user='joe@localhost')
     expected = {
       u'request': {
         u'authenticated': u'user:user@example.com',
         u'created_ts': str_now_30,
         u'expiration_secs': u'30',
         u'name': u'job1',
         u'priority': u'200',
         u'properties': {
@@ skipping 11 matching lines @@
           },
           u'command': [u'python', u'run_test.py'],
           u'dimensions': [
             {u'key': u'os', u'value': u'Amiga'},
             {u'key': u'pool', u'value': u'default'},
           ],
           u'execution_timeout_secs': u'3600',
           u'grace_period_secs': u'30',
           u'idempotent': True,
           u'io_timeout_secs': u'1200',
+          u'outputs': [u'foo', u'path/to/foobar'],
         },
         u'service_account': u'none',
         u'tags': [
           u'foo:bar',
           u'os:Amiga',
           u'pool:default',
           u'priority:200',
           u'service_account:none',
           u'user:joe@localhost',
         ],
@@ skipping 125 matching lines @@
             },
             u'command': [u'python', u'run_test.py'],
             u'dimensions': [
               {u'key': u'os', u'value': u'Amiga'},
               {u'key': u'pool', u'value': u'default'},
             ],
             u'execution_timeout_secs': u'3600',
             u'grace_period_secs': u'30',
             u'idempotent': True,
             u'io_timeout_secs': u'1200',
+            u'outputs': [u'foo', u'path/to/foobar'],
           },
           u'service_account': u'none',
           u'tags': [
             u'foo:bar',
             u'os:Amiga',
             u'pool:default',
             u'priority:200',
             u'service_account:none',
             u'user:joe@localhost',
           ],
@@ skipping 20 matching lines @@
             },
             u'command': [u'python', u'run_test.py'],
             u'dimensions': [
               {u'key': u'os', u'value': u'Amiga'},
               {u'key': u'pool', u'value': u'default'},
             ],
             u'execution_timeout_secs': u'3600',
             u'grace_period_secs': u'30',
             u'idempotent': True,
             u'io_timeout_secs': u'1200',
+            u'outputs': [u'foo', u'path/to/foobar'],
           },
           u'service_account': u'none',
           u'tags': [
             u'commit:post',
             u'os:Amiga',
             u'os:Win',
             u'pool:default',
             u'priority:100',
             u'project:yay',
             u'service_account:none',
@@ skipping 909 matching lines @@
         },
         u'command': [u'python', u'run_test.py'],
         u'dimensions': [
           {u'key': u'os', u'value': u'Amiga'},
           {u'key': u'pool', u'value': u'default'},
         ],
         u'execution_timeout_secs': u'3600',
         u'grace_period_secs': u'30',
         u'idempotent': False,
         u'io_timeout_secs': u'1200',
+        u'outputs': [u'foo', u'path/to/foobar'],
       },
       u'service_account': u'none',
       u'tags': [
         u'os:Amiga',
         u'pool:default',
         u'priority:100',
         u'service_account:none',
         u'user:joe@localhost',
       ],
       u'user': u'joe@localhost',
@@ skipping 553 matching lines @@
     self.call_api('terminate', body={'bot_id': 'bot1'}, status=403)
 
 
 if __name__ == '__main__':
   if '-v' in sys.argv:
     unittest.TestCase.maxDiff = None
     logging.basicConfig(level=logging.DEBUG)
   else:
     logging.basicConfig(level=logging.CRITICAL)
   unittest.main()