[test] Switch off perf data feature on swarming.

[test] Switch off perf data feature on swarming.

The feature sometimes prevents subsequent swarming bots to
delete the work directory on windows.

The data file is not kept on swarming bots anyways, therefore
this switches off the feature completely.

BUG=chromium:535160
LOG=n

Committed: https://crrev.com/f39d1cd4aa1c790027783484b950b2a860dba849
Cr-Commit-Position: refs/heads/master@{#32168}

[Michael Achenbach, 2015-11-23 08:44:36]

Description was changed from

==========
[test] Switch off perf data feature on swarming.

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

to

==========
[test] Switch off perf data feature on swarming.

BUG=chromium:535160
LOG=n
==========

[Michael Achenbach, 2015-11-23 08:48:05]

Description was changed from

==========
[test] Switch off perf data feature on swarming.

BUG=chromium:535160
LOG=n
==========

to

==========
[test] Switch off perf data feature on swarming.

The feature sometimes prevents subsequent swarming bots to
delete the work directory on windows.

The data file is not kept on swarming bots anyways, therefore
this switches off the feature completely.

BUG=chromium:535160
LOG=n
==========

[Michael Achenbach, 2015-11-23 08:48:05]

machenbach@chromium.org changed reviewers:
+ jkummerow@chromium.org

[Michael Achenbach, 2015-11-23 08:48:14]

PTAL

[Michael Achenbach, 2015-11-23 08:51:50]

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

[commit-bot: I haz the power, 2015-11-23 08:51:56]

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

[Jakob Kummerow, 2015-11-23 08:52:21]

lgtm

[Michael Achenbach, 2015-11-23 09:07:03]

The CQ bit was unchecked by machenbach@chromium.org

[Michael Achenbach, 2015-11-23 09:07:45]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2015-11-23 09:07:47]

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

[commit-bot: I haz the power, 2015-11-23 09:10:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-23 09:11:24]

Patchset 2 (id:??) landed as
https://crrev.com/f39d1cd4aa1c790027783484b950b2a860dba849
Cr-Commit-Position: refs/heads/master@{#32168}

[kjellander_chromium, 2015-11-23 11:40:35]

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

[kjellander_chromium, 2015-11-23 11:40:36]

If "perf data feature" keeps locks on files that causes trouble for other
consumers, maybe it would be better to track that down and fix?

[Michael Achenbach, 2015-11-23 12:34:33]

On 2015/11/23 11:40:36, kjellander (chromium) wrote:
> If "perf data feature" keeps locks on files that causes trouble for other
> consumers, maybe it would be better to track that down and fix?

It happens on windows only and very very rarely. I don't have cycles right now
to dig for this... The suspicious comment is:
https://code.google.com/p/chromium/codesearch#chromium/src/v8/tools/testrunne...

But again: On swarming we can't use the perf data feature anyways as we can't
persist the database file. So I won't fix it for swarming.