New issue
Advanced search Search tips

Issue 693714 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: Feb 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 612740

Blocking:
issue 650432


Participants' hotlists:
PGO-bugs


Sign in to add a comment

Use Swarming to speed-up the PGO profiling benchmarks

Project Member Reported by sebmarchand@chromium.org, Feb 17 2017

Issue description

We’re currently running all the profiling benchmarks sequentially on the same bot, we could make this significantly faster by switching to use swarming instead. 
Using Swarming will allow us to run the different benchmarks in parallel on a pool of slaves, and then we’ll collect the profiles data from these slaves and we’ll merge then on the main builder.

 

Comment 1 by mar...@chromium.org, Feb 17 2017

Blockedon: 612740
Project Member

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

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3c96661cd7d066e3f4d6fe8b9760661ff76ea005

commit 3c96661cd7d066e3f4d6fe8b9760661ff76ea005
Author: catapult-deps-roller <catapult-deps-roller@chromium.org>
Date: Wed Feb 22 09:17:23 2017

Roll src/third_party/catapult/ d9036c9b7..e56eec861 (13 commits).

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/d9036c9b7e3d..e56eec86141a

$ git log d9036c9b7..e56eec861 --date=short --no-merges --format='%ad %ae %s'
2017-02-21 nednguyen Revert of [Telemetry] Migrate browser_test_runner to use typ as the test runner (patchset #1 id:1 of https://codereview.chromium.org/2701253008/ )
2017-02-21 jbudorick [devil] Add an iteration timeout option to cmd_helper.IterCmdStdoutLines.
2017-02-21 benjhayden Update style in value/ui/.
2017-02-21 benjhayden Dedent CSS in histogram_set_view.
2017-02-21 benjhayden Fix merged relationships in histogram-set-table.
2017-02-21 benjhayden Merge sample diagnostics.
2017-02-21 nednguyen Reland of [Telemetry] Migrate browser_test_runner to use typ as the test runner (patchset #1 id:1 of https://codereview.chromium.org/2700263002/ )
2017-02-21 benjhayden Update style in tab_view.html.
2017-02-21 benjhayden Allow toggling merged-telemetry-info-span.
2017-02-21 benjhayden Document createScalarSpan.
2017-02-21 charliea Update the update_v8 script and roll d8 on Windows to stable
2017-02-21 perezju [Devil] Improve error message for pull failure
2017-02-21 sebmarchand Use pgosweep.exe from the build directory if it's available

Created with:
  roll-dep src/third_party/catapult
BUG= 636153 , 693203 , 636153 , 693714 

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=catapult-sheriff@chromium.org

Review-Url: https://codereview.chromium.org/2710903002
Cr-Commit-Position: refs/heads/master@{#451952}

[modify] https://crrev.com/3c96661cd7d066e3f4d6fe8b9760661ff76ea005/DEPS

Lowering the priority of the PGO bugs as we've switched the official build to Clang. I'll close these bugs once we decide that we won't support the VS build anymore.
Status: WontFix (was: Untriaged)
WontFix'ing this issue as the switch to Clang make it obsolete.

Sign in to add a comment