task_runner: always use run_isolated

task_runner: always use run_isolated

run_isolated learned to run non-isolated commands. Use it for ordinary
commands in task_runner.

R=maruel@chromium.org
BUG=601022
Committed: https://github.com/luci/luci-py/commit/5c6162e91858e17f4cfcedb50438d2e2baf5ec12

[nodir, 2016-05-03 17:24:12]

PTAL

[nodir, 2016-05-03 17:24:18]

The CQ bit was checked by nodir@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-03 17:24:27]

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

[nodir, 2016-05-03 17:24:43]

Perhaps this code must be updated with isolate.py in the same CL?

[M-A Ruel, 2016-05-03 17:48:57]

On 2016/05/03 17:24:43, nodir wrote:
> Perhaps this code must be updated with isolate.py in the same CL?

isolate.py?

The CL itself lgtm. Deploying to chros prox will require more careful
deployment.

[nodir, 2016-05-03 17:54:16]

On 2016/05/03 17:48:57, M-A Ruel wrote:
> On 2016/05/03 17:24:43, nodir wrote:
> > Perhaps this code must be updated with isolate.py in the same CL?
> 
> isolate.py?

Sorry, I meant CMDreproduce in swarming.py

[nodir, 2016-05-03 17:54:56]

I've removed hard timeout too, ptal

[nodir, 2016-05-09 22:51:06]

ping

[M-A Ruel, 2016-05-10 14:17:16]

lgtm

[nodir, 2016-05-11 16:15:48]

The CQ bit was checked by nodir@chromium.org

[commit-bot: I haz the power, 2016-05-11 16:15:58]

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

[commit-bot: I haz the power, 2016-05-11 17:39:51]

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

[commit-bot: I haz the power, 2016-05-11 17:39:52]

Try jobs failed on following builders:
  Luci-py Presubmit on master.tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Luci-py%20Presubmit/bui...)

[nodir, 2016-05-12 21:39:37]

Patchset #3 (id:40001) has been deleted

[nodir, 2016-05-12 21:41:23]

Patchset #3 (id:60001) has been deleted

[nodir, 2016-05-12 21:43:14]

Patchset #3 (id:80001) has been deleted

[nodir, 2016-05-12 21:46:23]

PTAL again. This change is not that easy

https://codereview.chromium.org/1949613002/diff/100001/appengine/swarming/swa...
File appengine/swarming/swarming_bot/bot_code/task_runner_test.py (left):

https://codereview.chromium.org/1949613002/diff/100001/appengine/swarming/swa...
appengine/swarming/swarming_bot/bot_code/task_runner_test.py:370:
self.assertEqual(task_details.command, cmd)
this test is not about checking command anyway

https://codereview.chromium.org/1949613002/diff/100001/appengine/swarming/swa...
appengine/swarming/swarming_bot/bot_code/task_runner_test.py:1041: expected = {
wasn't used

[nodir, 2016-05-12 21:52:08]

The CQ bit was checked by nodir@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-12 21:52:18]

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

[commit-bot: I haz the power, 2016-05-12 21:59:09]

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

[commit-bot: I haz the power, 2016-05-12 21:59:09]

Dry run: Try jobs failed on following builders:
  Luci-py Presubmit on master.tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Luci-py%20Presubmit/bui...)

[M-A Ruel, 2016-05-13 00:09:28]

lgtm

[nodir, 2016-05-25 00:24:52]

The CQ bit was checked by nodir@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-25 00:25:02]

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

[commit-bot: I haz the power, 2016-05-25 00:30:32]

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

[commit-bot: I haz the power, 2016-05-25 00:30:33]

Dry run: Try jobs failed on following builders:
  Luci-py Presubmit on master.tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Luci-py%20Presubmit/bui...)

[nodir, 2016-05-27 17:04:53]

The CQ bit was checked by nodir@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-27 17:05:06]

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

[commit-bot: I haz the power, 2016-05-27 17:14:32]

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

[commit-bot: I haz the power, 2016-05-27 17:14:34]

Dry run: Try jobs failed on following builders:
  Luci-py Presubmit on master.tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Luci-py%20Presubmit/bui...)

[nodir, 2016-05-27 18:17:58]

Patchset #7 (id:180001) has been deleted

[nodir, 2016-05-27 18:44:20]

The CQ bit was checked by nodir@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-27 18:44:26]

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

[nodir, 2016-05-27 18:45:41]

Patchset #7 (id:200001) has been deleted

[nodir, 2016-05-27 18:47:36]

The CQ bit was checked by nodir@chromium.org to run a CQ dry run

[nodir, 2016-05-27 18:47:39]

Patchset #7 (id:220001) has been deleted

[commit-bot: I haz the power, 2016-05-27 18:47:46]

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

[commit-bot: I haz the power, 2016-05-27 18:54:15]

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

[commit-bot: I haz the power, 2016-05-27 18:54:16]

Dry run: This issue passed the CQ dry run.

[nodir, 2016-05-27 20:28:27]

The CQ bit was checked by nodir@chromium.org

[nodir, 2016-05-27 20:28:27]

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/1949613002/#ps260001
(title: "rebased")

[commit-bot: I haz the power, 2016-05-27 20:28:38]

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

[nodir, 2016-05-27 20:29:02]

The CQ bit was unchecked by nodir@chromium.org

[nodir, 2016-05-27 20:31:11]

Patchset #8 (id:260001) has been deleted

[nodir, 2016-05-27 20:31:19]

Patchset #7 (id:240001) has been deleted

[nodir, 2016-05-27 20:31:39]

The CQ bit was checked by nodir@chromium.org

[nodir, 2016-05-27 20:31:40]

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/1949613002/#ps280001
(title: "rebased and ignored bot_overhead")

[commit-bot: I haz the power, 2016-05-27 20:31:45]

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

[commit-bot: I haz the power, 2016-05-27 20:33:53]

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

[commit-bot: I haz the power, 2016-05-27 20:33:55]

Try jobs failed on following builders:
  Luci-py Presubmit on master.tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Luci-py%20Presubmit/bui...)

[nodir, 2016-05-27 20:50:02]

The CQ bit was checked by nodir@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-27 20:50:14]

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

[commit-bot: I haz the power, 2016-05-27 20:53:03]

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

[commit-bot: I haz the power, 2016-05-27 20:53:05]

Dry run: This issue passed the CQ dry run.

[nodir, 2016-05-27 20:53:30]

The CQ bit was checked by nodir@chromium.org

[nodir, 2016-05-27 20:53:30]

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/1949613002/#ps300001
(title: "rebased")

[commit-bot: I haz the power, 2016-05-27 20:53:33]

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

[commit-bot: I haz the power, 2016-05-27 20:59:10]

Description was changed from

==========
task_runner: always use run_isolated

run_isolated learned to run non-isolated commands. Use it for ordinary
commands in task_runner.

R=maruel@chromium.org
BUG=601022
==========

to

==========
task_runner: always use run_isolated

run_isolated learned to run non-isolated commands. Use it for ordinary
commands in task_runner.

R=maruel@chromium.org
BUG=601022

Committed:
https://github.com/luci/luci-py/commit/5c6162e91858e17f4cfcedb50438d2e2baf5ec12
==========

[commit-bot: I haz the power, 2016-05-27 20:59:11]

Committed patchset #8 (id:300001) as
https://github.com/luci/luci-py/commit/5c6162e91858e17f4cfcedb50438d2e2baf5ec12