[devil] Set cache token only after successful initialization.

[devil] Set cache token only after successful initialization.

BUG=chromium:600047
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/3bc096f91bc5900d5712847c5bbb1b6e6ba30d95

[jbudorick, 2016-04-04 19:58:08]

jbudorick@chromium.org changed reviewers:
+ agrieve@chromium.org

[agrieve, 2016-04-04 20:00:15]

On 2016/04/04 19:58:09, jbudorick wrote:

lgtm

[jbudorick, 2016-04-04 20:01:08]

https://codereview.chromium.org/1860663002/diff/1/devil/devil/utils/run_tests...
File devil/devil/utils/run_tests_helper.py (right):

https://codereview.chromium.org/1860663002/diff/1/devil/devil/utils/run_tests...
devil/devil/utils/run_tests_helper.py:33: def SetLogLevel(verbose_count,
formatter=None):
er, this snuck in from another CL. removing before landing

[jbudorick, 2016-04-04 20:05:01]

Description was changed from

==========
[devil] Set cache token only after successful initialization.

BUG=chromium:#600047
==========

to

==========
[devil] Set cache token only after successful initialization.

BUG=chromium:600047
==========

[jbudorick, 2016-04-04 20:06:19]

The CQ bit was checked by jbudorick@chromium.org

[jbudorick, 2016-04-04 20:06:20]

The patchset sent to the CQ was uploaded after l-g-t-m from agrieve@chromium.org
Link to the patchset: https://codereview.chromium.org/1860663002/#ps20001
(title: "remove errant run_tests_helper change")

[commit-bot: I haz the power, 2016-04-04 20:06:26]

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

[jbudorick, 2016-04-04 20:22:25]

The CQ bit was unchecked by jbudorick@chromium.org

[jbudorick, 2016-04-04 21:11:24]

The CQ bit was checked by jbudorick@chromium.org

[jbudorick, 2016-04-04 21:11:25]

The patchset sent to the CQ was uploaded after l-g-t-m from agrieve@chromium.org
Link to the patchset: https://codereview.chromium.org/1860663002/#ps40001
(title: "+tests")

[commit-bot: I haz the power, 2016-04-04 21:11:30]

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

[commit-bot: I haz the power, 2016-04-04 21:24:47]

Description was changed from

==========
[devil] Set cache token only after successful initialization.

BUG=chromium:600047
==========

to

==========
[devil] Set cache token only after successful initialization.

BUG=chromium:600047

Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2016-04-04 21:24:48]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...