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

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

BUG=

Committed: https://github.com/luci/luci-py/commit/7daf7ab4d5021ab8880770de452377cb36f0c36a

[aludwin, 2016-10-26 18:40:05]

aludwin@google.com changed reviewers:
+ maruel@google.com

[M-A Ruel, 2016-10-26 18:47:31]

maruel@chromium.org changed reviewers:
+ maruel@chromium.org

[M-A Ruel, 2016-10-26 18:47:32]

+Kevin in practice we'll want to show them in the UI.

The only blocker is the verification of the paths.

https://codereview.chromium.org/2453873002/diff/20001/appengine/swarming/serv...
File appengine/swarming/server/task_request.py (right):

https://codereview.chromium.org/2453873002/diff/20001/appengine/swarming/serv...
appengine/swarming/server/task_request.py:425: # will be added to those in that
directory.
# The path must be in posix form.

You must add a validator. See the path validator above for example.

https://codereview.chromium.org/2453873002/diff/20001/client/swarming.py
File client/swarming.py (right):

https://codereview.chromium.org/2453873002/diff/20001/client/swarming.py#newc...
client/swarming.py:8: __version__ = '0.8.7'
bump

[aludwin, 2016-10-27 18:28:34]

I manually confirmed the feature works on the latest PS

[M-A Ruel, 2016-10-27 20:05:50]

lgtm

[aludwin, 2016-10-27 22:04:43]

You can ignore this - just rebased

[aludwin, 2016-10-27 22:04:46]

You can ignore this - just rebased

[M-A Ruel, 2016-10-27 22:22:44]

lgtm

[aludwin, 2016-10-27 22:48:04]

The CQ bit was checked by aludwin@google.com

[commit-bot: I haz the power, 2016-10-27 22:48:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-27 22:48:15]

This CL has an open dependency (Issue 2440353004 Patch 160001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[aludwin, 2016-10-27 22:59:31]

The CQ bit was checked by aludwin@google.com

[commit-bot: I haz the power, 2016-10-27 22:59:35]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-27 23:06:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-27 23:06:20]

Try jobs failed on following builders:
  Luci-py Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/322130639c4c6910)

[aludwin, 2016-10-28 02:46:00]

The CQ bit was checked by aludwin@google.com

[aludwin, 2016-10-28 02:46:00]

The patchset sent to the CQ was uploaded after l-g-t-m from maruel@chromium.org
Link to the patchset: https://codereview.chromium.org/2453873002/#ps100001
(title: "Fix broken smoke test and rebase")

[commit-bot: I haz the power, 2016-10-28 02:46:06]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[aludwin, 2016-10-28 02:47:21]

The CQ bit was unchecked by aludwin@google.com

[aludwin, 2016-10-28 02:50:19]

The CQ bit was checked by aludwin@google.com

[aludwin, 2016-10-28 02:50:19]

The patchset sent to the CQ was uploaded after l-g-t-m from maruel@chromium.org
Link to the patchset: https://codereview.chromium.org/2453873002/#ps120001
(title: "Fix rebase mistake (unnecessary import)")

[commit-bot: I haz the power, 2016-10-28 02:50:26]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-28 02:51:55]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-28 02:51:56]

Try jobs failed on following builders:
  Luci-py Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/3222045a6d208410)

[aludwin, 2016-10-28 02:54:27]

The CQ bit was checked by aludwin@google.com

[commit-bot: I haz the power, 2016-10-28 02:54:36]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-28 02:56:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-28 02:56:07]

Try jobs failed on following builders:
  Luci-py Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/3222082c4330c810)

[aludwin, 2016-10-28 03:21:46]

The CQ bit was checked by aludwin@google.com

[aludwin, 2016-10-28 03:21:47]

The patchset sent to the CQ was uploaded after l-g-t-m from maruel@chromium.org
Link to the patchset: https://codereview.chromium.org/2453873002/#ps140001
(title: "Extra help text for examples")

[commit-bot: I haz the power, 2016-10-28 03:21:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-28 03:23:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-28 03:23:23]

Try jobs failed on following builders:
  Luci-py Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/322221237877a610)

[aludwin, 2016-10-28 16:50:44]

The CQ bit was checked by aludwin@google.com

[commit-bot: I haz the power, 2016-10-28 16:50:51]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-28 17:25:26]

Description was changed from

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

BUG=
==========

to

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

BUG=

Committed:
https://github.com/luci/luci-py/commit/7daf7ab4d5021ab8880770de452377cb36f0c36a
==========

[commit-bot: I haz the power, 2016-10-28 17:25:27]

Committed patchset #8 (id:140001) as
https://github.com/luci/luci-py/commit/7daf7ab4d5021ab8880770de452377cb36f0c36a