Swarming tasks: print start and end time

Swarming tasks: print start and end time

This will help determine how long each component of the task takes.

NOTRY=true
BUG=skia:4763
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1769583002

Committed: https://skia.googlesource.com/skia/+/d4f89e4e391d00e1011e3bbafc90743335e40030

[borenet, 2016-03-04 18:18:36]

The CQ bit was checked by borenet@google.com to run a CQ dry run

[borenet, 2016-03-04 18:18:39]

Description was changed from

==========
Swarming tasks: print start and end time

This will help determine how long each component of the task takes.

NOTRY=true
BUG=skia:4763
==========

to

==========
Swarming tasks: print start and end time

This will help determine how long each component of the task takes.

NOTRY=true
BUG=skia:4763
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[commit-bot: I haz the power, 2016-03-04 18:18:42]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1769583002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1769583002/1

[borenet, 2016-03-04 18:19:25]

borenet@google.com changed reviewers:
+ rmistry@google.com

[commit-bot: I haz the power, 2016-03-04 18:20:12]

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

[commit-bot: I haz the power, 2016-03-04 18:20:13]

Dry run: Try jobs failed on following builders:
  skia_presubmit-Trybot on client.skia.fyi (JOB_FAILED,
http://build.chromium.org/p/client.skia.fyi/builders/skia_presubmit-Trybot/bu...)

[borenet, 2016-03-04 18:20:37]

The CQ bit was checked by borenet@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-03-04 18:20:42]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1769583002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1769583002/20001

[rmistry, 2016-03-04 18:20:59]

https://codereview.chromium.org/1769583002/diff/1/infra/bots/compile_skia.py
File infra/bots/compile_skia.py (right):

https://codereview.chromium.org/1769583002/diff/1/infra/bots/compile_skia.py#...
infra/bots/compile_skia.py:24: print 'Task finished at %s UTC' %
time.asctime(time.gmtime())
Why not calculate current-time - start-time to make it easy to figure out how
long it took.

[commit-bot: I haz the power, 2016-03-04 18:21:53]

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

[commit-bot: I haz the power, 2016-03-04 18:21:54]

Dry run: This issue passed the CQ dry run.

[borenet, 2016-03-04 18:48:03]

The CQ bit was checked by borenet@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-03-04 18:48:09]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1769583002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1769583002/40001

[commit-bot: I haz the power, 2016-03-04 18:49:20]

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

[commit-bot: I haz the power, 2016-03-04 18:49:21]

Dry run: This issue passed the CQ dry run.

[borenet, 2016-03-04 18:50:01]

Sigh. I miss Go's time package.

https://codereview.chromium.org/1769583002/diff/1/infra/bots/compile_skia.py
File infra/bots/compile_skia.py (right):

https://codereview.chromium.org/1769583002/diff/1/infra/bots/compile_skia.py#...
infra/bots/compile_skia.py:24: print 'Task finished at %s UTC' %
time.asctime(time.gmtime())
On 2016/03/04 18:20:59, rmistry wrote:
> Why not calculate current-time - start-time to make it easy to figure out how
> long it took.

Done, but I really want both the start and finish times printed so that I can
determine how much time passed between the task starting and this script
beginning to run (ie. time spent downloading inputs) and the time between this
script finishing and the task finishing (ie. time spent uploading outputs).

[rmistry, 2016-03-04 18:53:26]

lgtm

https://codereview.chromium.org/1769583002/diff/40001/infra/bots/perf_skia.py
File infra/bots/perf_skia.py (right):

https://codereview.chromium.org/1769583002/diff/40001/infra/bots/perf_skia.py...
infra/bots/perf_skia.py:13: import time
unused

[borenet, 2016-03-04 18:57:06]

The CQ bit was checked by borenet@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-03-04 18:57:13]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1769583002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1769583002/60001

[borenet, 2016-03-04 18:57:24]

https://codereview.chromium.org/1769583002/diff/40001/infra/bots/perf_skia.py
File infra/bots/perf_skia.py (right):

https://codereview.chromium.org/1769583002/diff/40001/infra/bots/perf_skia.py...
infra/bots/perf_skia.py:13: import time
On 2016/03/04 18:53:25, rmistry wrote:
> unused

Done.

[borenet, 2016-03-04 18:57:40]

The CQ bit was unchecked by borenet@google.com

[borenet, 2016-03-04 18:57:44]

The CQ bit was checked by borenet@google.com

[borenet, 2016-03-04 18:57:45]

The patchset sent to the CQ was uploaded after l-g-t-m from rmistry@google.com
Link to the patchset: https://codereview.chromium.org/1769583002/#ps60001
(title: "Fix unused import")

[commit-bot: I haz the power, 2016-03-04 18:57:54]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1769583002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1769583002/60001

[commit-bot: I haz the power, 2016-03-04 18:58:23]

Description was changed from

==========
Swarming tasks: print start and end time

This will help determine how long each component of the task takes.

NOTRY=true
BUG=skia:4763
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Swarming tasks: print start and end time

This will help determine how long each component of the task takes.

NOTRY=true
BUG=skia:4763
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/d4f89e4e391d00e1011e3bbafc90743335e40030
==========

[commit-bot: I haz the power, 2016-03-04 18:58:24]

Committed patchset #4 (id:60001) as
https://skia.googlesource.com/skia/+/d4f89e4e391d00e1011e3bbafc90743335e40030