Switch swarming.py to use the new client API, add flags, print more info.

Switch swarming.py to use the new client API, add flags, print more info.

- Rename --deadline to --expiration.
- Add --user, --io-timeout, --hard-timeout and --tags.
- The switch to the new API will permit to get rid of the old client API on the
  server.
- Print url in "swarming.py trigger" for easier copy-paste, in addition to the
  copy-pasteable command. Useful when looking at the logs in a web browser.
- Fix swarming_smoke_test.py which regressed since 4566ecf76b.

R=vadimsh@chromium.org
BUG=swarming:118

Committed: https://code.google.com/p/swarming/source/detail?repo=client&r=2f6581a2479ec3d404c110a6ea2ff6f8b8e081ed

[M-A, 2014-10-01 17:28:22]

After that, it'll be easier to add more features.

[vadimsh, 2014-10-01 19:41:23]

lgtm with nits

my main concern is 'unknown' user

https://codereview.appspot.com/145020044/diff/80001/swarming.py
File swarming.py (right):

https://codereview.appspot.com/145020044/diff/80001/swarming.py#newcode405
swarming.py:405: # TODO(maruel): Get rid of this.
Any plans how to get rid of it?

https://codereview.appspot.com/145020044/diff/80001/swarming.py#newcode574
swarming.py:574: 'user': user or 'unknown',
Can we pass None instead of 'unknown'?

For tasks triggered by developers server can replace None with
auth.get_current_identity(). 'unknown' seems too arbitrary.

https://codereview.appspot.com/145020044/diff/80001/swarming.py#newcode816
swarming.py:816: '--user', help='User associated to the task.')
nit: associated with the task

https://codereview.appspot.com/145020044/diff/80001/swarming.py#newcode822
swarming.py:822: '--hard-timeout', type='int', default=60*60,
Make this TOP_LEVEL_CONSTANT, or remove DEFAULT_SHARD_WAIT_TIME contanst (in
favor of specifying value explicitly). I don't like the inconsistency.

[M-A, 2014-10-01 20:38:28]

https://codereview.appspot.com/145020044/diff/80001/swarming.py
File swarming.py (right):

https://codereview.appspot.com/145020044/diff/80001/swarming.py#newcode405
swarming.py:405: # TODO(maruel): Get rid of this.
On 2014/10/01 19:41:22, vadimsh wrote:
> Any plans how to get rid of it?

No firm plan yet. The goal would be to shave off a few seconds off triggers.

It would likely end up having run_isolated.zip generated on the server the same
way swarming_bot.zip is generated so it wouldn't need to be constantly uploaded.

https://codereview.appspot.com/145020044/diff/80001/swarming.py#newcode574
swarming.py:574: 'user': user or 'unknown',
On 2014/10/01 19:41:22, vadimsh wrote:
> Can we pass None instead of 'unknown'?
> 
> For tasks triggered by developers server can replace None with
> auth.get_current_identity(). 'unknown' seems too arbitrary.

Done.

https://codereview.appspot.com/145020044/diff/80001/swarming.py#newcode816
swarming.py:816: '--user', help='User associated to the task.')
On 2014/10/01 19:41:22, vadimsh wrote:
> nit: associated with the task

Done.

https://codereview.appspot.com/145020044/diff/80001/swarming.py#newcode822
swarming.py:822: '--hard-timeout', type='int', default=60*60,
On 2014/10/01 19:41:22, vadimsh wrote:
> Make this TOP_LEVEL_CONSTANT, or remove DEFAULT_SHARD_WAIT_TIME contanst (in
> favor of specifying value explicitly). I don't like the inconsistency.

Done.

[vadimsh, 2014-10-01 23:50:46]

lgtm with nits

https://codereview.appspot.com/145020044/diff/100001/auth.py
File auth.py (right):

https://codereview.appspot.com/145020044/diff/100001/auth.py#newcode92
auth.py:92: return
nit: return None

https://codereview.appspot.com/145020044/diff/100001/swarming.py
File swarming.py (right):

https://codereview.appspot.com/145020044/diff/100001/swarming.py#newcode845
swarming.py:845: default=80 * 60.,
nit: 80*60, to match style used above

[M-A, 2014-10-03 15:09:32]

https://codereview.appspot.com/145020044/diff/100001/auth.py
File auth.py (right):

https://codereview.appspot.com/145020044/diff/100001/auth.py#newcode92
auth.py:92: return
On 2014/10/01 23:50:46, vadimsh wrote:
> nit: return None

Done.

https://codereview.appspot.com/145020044/diff/100001/swarming.py
File swarming.py (right):

https://codereview.appspot.com/145020044/diff/100001/swarming.py#newcode845
swarming.py:845: default=80 * 60.,
On 2014/10/01 23:50:46, vadimsh wrote:
> nit: 80*60, to match style used above

Done.

[M-A, 2014-10-03 15:10:18]

Committed patchset #3 (id:120001) manually as
2f6581a2479ec3d404c110a6ea2ff6f8b8e081ed (presubmit successful).