New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 676080 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 688416



Sign in to add a comment

Do we remove Nexus 9 devices from *chromium.perf*?

Project Member Reported by benhenry@google.com, Dec 20 2016

Issue description

1. We have tablet representation on the waterfalls.
2. 9s are buggy and expensive to deal with right now
3. There may be other problems that I'm forgetting right now.
 
The biggest arguments I can think of for keeping the N9 around are:
* Additional 64 bit coverage
* Tablet coverage on L (N7v2 is on K)

Screen resolution doesn't seem to be a huge win, N7v2 is 1920×1200 and N9 is 2048x1536

The biggest arguments I cn think of for removing:
* They're buggy and can't be replaced
* Currently the android developer dashboard says 6.4% of users have "large" screens and 3.4% have "x-large", which doesn't justify two tablets on the waterfall IMO: https://developer.android.com/about/dashboards/index.html
GPU is using Pixel Cs. They may be an alternative if we want 64-bit, L+ tablet coverage and can't tolerate the N9s.

Comment 3 by benhenry@google.com, Dec 21 2016

Owner: benhenry@chromium.org
Status: Assigned (was: Untriaged)
I heard some arguments a few weeks ago but haven't heard much since then. I'll take this on and ping a couple more times before closing this.
Ok, let's do this.

What are the nexus9 devices called? I can now find all of them here: https://docs.google.com/spreadsheets/d/1ZpjJ1dmXhOVY6DfcCcDmPcjWD5wPy60nrjdKDN-ynhE/edit#gid=1129474336
Android devices to recover:

HT4CTJT03744
HT4CTJT03478
HT4B6JT02453
HT4B6JT02626
HT4B7JT01220
HT4BPJT01045
HT4BRJT00749
HT52SJT00009
HT52SJT00121
HT4CTJT04297
HT4B6JT02450
HT4B6JT02657
HT4B7JT01059
HT4B7JT01232
HT4B7JT01243
HT4B7JT01277
HT4B7JT01981
HT4CTJT03484
HT4CTJT03649
HT4CTJT03664
HT4CTJT03735
HT4CTJT04123
HT4CTJT04201
HT4CTJT04287

Builders/Hosts to reclaim:
build104-b4
build105-b4
build112-b4
build11-b1
build12-b1
build46-b1

CL is in review, will need to restart two masters before we reap.

Project Member

Comment 7 by bugdroid1@chromium.org, Feb 2 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build/+/54618c8f2c28b9391ff8c29a8c2fa002d2aaa5ed

commit 54618c8f2c28b9391ff8c29a8c2fa002d2aaa5ed
Author: Ben Henry <benhenry@google.com>
Date: Thu Feb 02 23:39:16 2017

Perf waterfalls: Removing all Nexus9s

BUG= 676080 
TEST=None

Change-Id: I9a87d49f90cdca75fa6d77d951cd50494fe8b716
Reviewed-on: https://chromium-review.googlesource.com/435435
Reviewed-by: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: Annie Sullivan <sullivan@chromium.org>
Commit-Queue: Ben Henry <benhenry@chromium.org>

[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect.expected/basic_perf_tryjob_with_metric_android_nexus9_perf_bisect.json
[modify] https://crrev.com/54618c8f2c28b9391ff8c29a8c2fa002d2aaa5ed/scripts/slave/recipes/android/perf.py
[modify] https://crrev.com/54618c8f2c28b9391ff8c29a8c2fa002d2aaa5ed/scripts/slave/recipes/chromium.expected/full_chromium_perf_Android_arm64_Compile.json
[modify] https://crrev.com/54618c8f2c28b9391ff8c29a8c2fa002d2aaa5ed/scripts/slave/recipes/bisection/android_bisect_staging.py
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/chromium.expected/full_chromium_perf_Android_Nexus9_Perf__1_.json
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect.expected/basic_perf_tryjob_android_nexus9_perf_bisect.json
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect.expected/perf_tryjob_config_error_android_nexus9_perf_bisect.json
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect.expected/perf_cq_no_benchmark_to_run_android_nexus9_perf_bisect.json
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Nexus9_Perf__3_.json
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect_staging.expected/basic_perf_tryjob_android_nexus9_perf_bisect.json
[modify] https://crrev.com/54618c8f2c28b9391ff8c29a8c2fa002d2aaa5ed/scripts/slave/recipes/bisection/android_bisect.py
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect_staging.expected/basic_perf_tryjob_with_revisions_android_nexus9_perf_bisect.json
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect_staging.expected/basic_recipe_android_nexus9_perf_bisect.json
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect_staging.expected/perf_cq_no_changes_android_nexus9_perf_bisect.json
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect_staging.expected/perf_tryjob_failed_test_android_nexus9_perf_bisect.json
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect_staging.expected/perf_cq_no_benchmark_to_run_android_nexus9_perf_bisect.json
[modify] https://crrev.com/54618c8f2c28b9391ff8c29a8c2fa002d2aaa5ed/masters/master.tryserver.chromium.perf/master.cfg
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/chromium.expected/full_chromium_perf_Android_Nexus9_Perf__2_.json
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/chromium.expected/full_chromium_perf_Android_Nexus9_Perf__3_.json
[modify] https://crrev.com/54618c8f2c28b9391ff8c29a8c2fa002d2aaa5ed/masters/master.tryserver.chromium.perf/slaves.cfg
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Nexus9_Perf__2_.json
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect_staging.expected/basic_perf_tryjob_with_metric_android_nexus9_perf_bisect.json
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect.expected/perf_tryjob_failed_test_android_nexus9_perf_bisect.json
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect_staging.expected/perf_tryjob_config_error_android_nexus9_perf_bisect.json
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect.expected/basic_recipe_android_nexus9_perf_bisect.json
[modify] https://crrev.com/54618c8f2c28b9391ff8c29a8c2fa002d2aaa5ed/masters/master.chromium.perf/slaves.cfg
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect.expected/basic_perf_tryjob_with_revisions_android_nexus9_perf_bisect.json
[modify] https://crrev.com/54618c8f2c28b9391ff8c29a8c2fa002d2aaa5ed/scripts/slave/recipe_modules/chromium_tests/chromium_perf.py
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Nexus9_Perf__1_.json
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect.expected/perf_cq_no_changes_android_nexus9_perf_bisect.json
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect.expected/perf_cq_run_benchmark_android_nexus9_perf_bisect.json
[delete] https://crrev.com/4f6095322007d2f0294dec333752ef2689b7f41b/scripts/slave/recipes/bisection/android_bisect_staging.expected/perf_cq_run_benchmark_android_nexus9_perf_bisect.json
[modify] https://crrev.com/54618c8f2c28b9391ff8c29a8c2fa002d2aaa5ed/masters/master.chromium.perf/master.cfg

Project Member

Comment 8 by bugdroid1@chromium.org, Feb 3 2017

Comment 9 by dtu@chromium.org, Feb 3 2017

Blocking: 688416
I'll remove these as an option from the dashboard.
Ben, feel free to assign this bug to me to reclaim the slaves and devices.
Owner: pschmidt@chromium.org
Do you need a list of hosts/devices?
Nah.  I can pick it up from that cl.
Cc: jasontiller@chromium.org
Thanks.

+jasontiller FYI
Project Member

Comment 15 by bugdroid1@chromium.org, Feb 6 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome-golo/chrome-golo/+/6cabd417b1f5933eec0a61bba22f4ea87f94847a

commit 6cabd417b1f5933eec0a61bba22f4ea87f94847a
Author: Peter Schmidt <pschmidt@google.com>
Date: Mon Feb 06 19:46:06 2017

Status: Fixed (was: Assigned)
Hosts and N9's have been reclaimed.
Cc: majidvp@chromium.org
 Issue 691040  has been merged into this issue.
I think if we are taking out a perf bot it will be nice to send a PSA to perf-sherrifs@chromium.org. Next sheriff will get a heads up when triaging data stoppage alerts.

Sign in to add a comment