Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(823)

Unified Diff: ct/go/worker_scripts/run_chromium_perf/main.go

Issue 1392173005: [CT] Update worker scripts to use new benchmarks and delete old benchmarks (Closed) Base URL: https://skia.googlesource.com/buildbot@master
Patch Set: Cleanup capture_archives Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ct/go/worker_scripts/run_chromium_perf/main.go
diff --git a/ct/go/worker_scripts/run_chromium_perf/main.go b/ct/go/worker_scripts/run_chromium_perf/main.go
index dc9661a05786ee66e055fddfd3e71e47398deeee..7282c4f602e31e597d860cacbe7adbac82035bc0 100644
--- a/ct/go/worker_scripts/run_chromium_perf/main.go
+++ b/ct/go/worker_scripts/run_chromium_perf/main.go
@@ -244,20 +244,22 @@ func runBenchmark(fileInfoName, pathToPagesets, pathToPyFiles, localOutputDir, c
return nil
}
- // Convert the filename into a format consumable by the run_benchmarks
- // binary.
+ // Read the pageset.
pagesetName := strings.TrimSuffix(pagesetBaseName, filepath.Ext(pagesetBaseName))
pagesetPath := filepath.Join(pathToPagesets, fileInfoName)
+ decodedPageset, err := util.ReadPageset(pagesetPath)
+ if err != nil {
+ return fmt.Errorf("Could not read %s: %s", pagesetPath, err)
+ }
glog.Infof("===== Processing %s for %s =====", pagesetPath, runID)
-
- skutil.LogErr(os.Chdir(pathToPyFiles))
args := []string{
- util.BINARY_RUN_BENCHMARK,
- fmt.Sprintf("%s.%s", *benchmarkName, util.BenchmarksToPagesetName[*benchmarkName]),
- "--page-set-name=" + pagesetName,
- "--page-set-base-dir=" + pathToPagesets,
+ filepath.Join(util.TelemetryBinariesDir, util.BINARY_RUN_BENCHMARK),
+ util.BenchmarksToPagesetName[*benchmarkName],
"--also-run-disabled-tests",
+ "--user-agent=" + decodedPageset.UserAgent,
+ "--urls-list=" + decodedPageset.UrlsList,
+ "--archive-data-file=" + decodedPageset.ArchiveDataFile,
}
// Need to capture output for all benchmarks.

Powered by Google App Engine
This is Rietveld 408576698