Merge Tests>Telemetry and Speed>Telemetry |
||||||
Issue descriptionNed, Emily, Juan, Could you please let me know if there is any reason to have both of these? Otherwise please respond with LGTM. If there is not, then I will find an admin who can merge them. (Sorry Ned, I know you're trying to get off of these Telemetry bugs, but you have the most history here. You can drop yourself from the CC list after you let me know whether we still need both for any reason.) Also, I don't really care whether we keep Tests>Telemetry or Speed>Telemetry in the end, but I'll put out as the straw man that we keep "Tests>Telemetry" (or "Test>Telemetry" if that is where we are going.)
,
Jan 11
Test>Telemetry is supposed for correctness testing stuffs, e.g: gpu tests. These are bugs related to the browser startup reliablity, how to simulate certain user interactions, the correctness harness Speed>Telemetry is for benchmarking/perf aspect. Example are bugs about whether we should collect garbage on the browser between story run in the benchmark loop. For that reason, we keep two separate bug component. Though with the new Chrome iniative of 1 team = 1 bug component, I support the decision to merge the two. Maybe we just need different bug labels to distinguish between the correctness testing & perf aspect of Telemetry
,
Jan 11
Thanks for the information, Ned! I think that since the rather small CCA is owning both functional and perf testing sides of Telemetry, it's fine to have just one component. The other thing is that if the component system isn't obvious to all the team members then people just won't follow it anyways, so I prefer a simple solution. Emily, Juan, please respond with LGTM unless you have some concern, and I will move forward.
,
Jan 14
Merge LGTM.
,
Jan 14
lgtm
,
Jan 14
Hi Ben, Do you think you could please move all the bugs that are currently in Tests>Telemetry and Speed>Telemetry into a new "Test>Telemetry" component? Note that we still have a Speed>Benchmarks component, but I guess we will leave that alone for now.
,
Jan 14
,
Jan 16
(6 days ago)
,
Jan 16
(6 days ago)
,
Jan 16
(6 days ago)
There are >3600 issues in Tests>Telemetry, so I'm only moving the 340 open issues. I've deprecated Tests>Telemetry. For Speed>Telemetry, I've moved all issues to Test>Telemetry and can delete the component. Also - LOL on the fact that I had to refresh this because of my changes.
,
Jan 16
(6 days ago)
Thanks Ben! |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by crouleau@chromium.org
, Jan 11