Perf: Removing S5s from waterfall

Perf: Removing S5s from waterfall

BUG=665142
TEST=NONE

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

[benhenry, 2016-11-16 01:22:32]

Description was changed from

==========
Perf: Removing S5s from waterfall

BUG=665142
TEST=NONE
==========

to

==========
Perf: Removing S5s from waterfall

BUG=665142
TEST=NONE
==========

[benhenry, 2016-11-16 01:22:32]

benhenry@google.com changed reviewers:
+ dtu@chromium.org, eyaich@google.com, jbudorick@chromium.org,
martiniss@chromium.org, stip@chromium.org

[dtu, 2016-11-16 01:30:19]

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

[commit-bot: I haz the power, 2016-11-16 01:30:52]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-16 01:45:26]

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

[commit-bot: I haz the power, 2016-11-16 01:45:26]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[benhenry, 2016-11-16 23:45:31]

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

[commit-bot: I haz the power, 2016-11-16 23:47:05]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-17 01:35:38]

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

[commit-bot: I haz the power, 2016-11-17 01:35:38]

Dry run: Try jobs failed on following builders:
  android_cronet on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_cron...)
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[benhenry, 2016-11-17 01:47:52]

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

[commit-bot: I haz the power, 2016-11-17 01:49:32]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-17 03:51:55]

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

[commit-bot: I haz the power, 2016-11-17 03:51:56]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_TIMED_OUT,
no build URL)
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_TIMED_OUT,
no build URL)
  android_compile_dbg on master.tryserver.chromium.android (JOB_TIMED_OUT, no
build URL)
  android_cronet on master.tryserver.chromium.android (JOB_TIMED_OUT, no build
URL)
  cast_shell_android on master.tryserver.chromium.android (JOB_TIMED_OUT, no
build URL)

[benhenry, 2016-11-17 17:39:50]

The CQ bit was checked by benhenry@google.com

[commit-bot: I haz the power, 2016-11-17 17:40:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-17 17:40:42]

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

[commit-bot: I haz the power, 2016-11-17 17:40:43]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[benhenry, 2016-11-17 23:21:57]

benhenry@google.com changed reviewers:
+ eakuefner@chromium.org

[benhenry, 2016-11-17 23:21:58]

PTAL

[eakuefner, 2016-11-17 23:22:56]

enthusiastic tools/perf OWNERS lgtm

[benhenry, 2016-11-17 23:24:00]

The CQ bit was checked by benhenry@google.com

[commit-bot: I haz the power, 2016-11-17 23:26:28]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-18 01:05:06]

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

[commit-bot: I haz the power, 2016-11-18 01:05:07]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[benhenry, 2016-11-18 01:07:58]

The CQ bit was checked by benhenry@google.com

[commit-bot: I haz the power, 2016-11-18 01:09:51]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-18 02:52:31]

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

[commit-bot: I haz the power, 2016-11-18 02:52:32]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[benhenry, 2016-11-18 17:55:59]

It looks like Dave is the only person with LGTM perms for
chromium.perf.fyi.json. That might need to be updated, but in the meantime -
Dave, PTAL.

[jbudorick, 2016-11-18 17:57:04]

On 2016/11/18 17:55:59, benhenry wrote:
> It looks like Dave is the only person with LGTM perms for
> chromium.perf.fyi.json. That might need to be updated, but in the meantime -
> Dave, PTAL.

The generic //testing/buildbot/ owners can also approve.

[dtu, 2016-11-18 17:57:51]

lgtm

[benhenry, 2016-11-18 18:02:57]

The CQ bit was checked by benhenry@google.com

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

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-18 20:08:47]

Description was changed from

==========
Perf: Removing S5s from waterfall

BUG=665142
TEST=NONE
==========

to

==========
Perf: Removing S5s from waterfall

BUG=665142
TEST=NONE
==========

[commit-bot: I haz the power, 2016-11-18 20:08:48]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-11-18 20:10:49]

Description was changed from

==========
Perf: Removing S5s from waterfall

BUG=665142
TEST=NONE
==========

to

==========
Perf: Removing S5s from waterfall

BUG=665142
TEST=NONE

Committed: https://crrev.com/c4f575f7106fce8f23a60e0a4f0ed17ddec485bf
Cr-Commit-Position: refs/heads/master@{#433276}
==========

[commit-bot: I haz the power, 2016-11-18 20:10:50]

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