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

Issue 2056623002: [Findit] Adding script for custom try job queries (Closed)

Created:
4 years, 6 months ago by lijeffrey
Modified:
4 years, 6 months ago
Reviewers:
chanli, stgao
CC:
chromium-reviews, infra-reviews+infra_chromium.org, Sharu Jiang
Base URL:
https://chromium.googlesource.com/infra/infra.git@master
Target Ref:
refs/heads/master
Project:
infra
Visibility:
Public.

Description

[Findit] Adding script for custom try job queries BUG=619637 Usage: python try_job_data_metrics.py [args] where args can be: -t: Separates try job by type (compile, test). -m: Aggregates data by each master (chromium.chrome, chromium.linux, etc) -b: Aggregates data by master and builder. Note passing this argument has the effect of breaking down by master first, then by builder. -r: Whether or not heuristic results were used to guide the try job -c: Whether or not compile targets were identified to speed up compile For example: python try_job_data_metrics.py try_jobs_per_day: ... average_regression_range_size: ... average_execution_time: ... average_commits_analyzed: ... longest_execution_time: ... shortest_execution_time: ... number_of_try_jobs: ... detection_rate: ... error_rate: ... time_per_revision: ... under_five_minutes_rate: ... under_fifteen_minutes_rate: ... under_thirty_minutes_rate: ... over_thirty_minutes_rate: ... python try_job_metrics.py -t Compile: <stats ...> Test: <stats ...> python try_job_metrics.py -t -m Compile: chromium.chrome: <stats ...> chromium.linux: <stats ...> ... Test: chromium.chrome: <stats ...> ... Committed: https://chromium.googlesource.com/infra/infra/+/0539a5da9122baf2b74dd6ea7684d9121e3f8b2b

Patch Set 1 #

Total comments: 2

Patch Set 2 : Addressing comments #

Patch Set 3 : Fixing gpylint issues #

Unified diffs Side-by-side diffs Delta from patch set Stats (+399 lines, -0 lines) Patch
A appengine/findit/util_scripts/remote_queries/try_job_data_metrics.py View 1 2 1 chunk +399 lines, -0 lines 0 comments Download

Messages

Total messages: 19 (10 generated)
lijeffrey
ptal
4 years, 6 months ago (2016-06-09 04:37:47 UTC) #3
chanli
On 2016/06/09 04:37:47, lijeffrey wrote: > ptal lgtm
4 years, 6 months ago (2016-06-13 22:46:11 UTC) #5
stgao
lgtm with a comment https://codereview.chromium.org/2056623002/diff/1/appengine/findit/util_scripts/remote_queries/try_job_data_metrics.py File appengine/findit/util_scripts/remote_queries/try_job_data_metrics.py (right): https://codereview.chromium.org/2056623002/diff/1/appengine/findit/util_scripts/remote_queries/try_job_data_metrics.py#newcode367 appengine/findit/util_scripts/remote_queries/try_job_data_metrics.py:367: args = sys.argv[1:] If we ...
4 years, 6 months ago (2016-06-13 23:48:21 UTC) #6
lijeffrey
https://codereview.chromium.org/2056623002/diff/1/appengine/findit/util_scripts/remote_queries/try_job_data_metrics.py File appengine/findit/util_scripts/remote_queries/try_job_data_metrics.py (right): https://codereview.chromium.org/2056623002/diff/1/appengine/findit/util_scripts/remote_queries/try_job_data_metrics.py#newcode367 appengine/findit/util_scripts/remote_queries/try_job_data_metrics.py:367: args = sys.argv[1:] On 2016/06/13 23:48:21, stgao wrote: > ...
4 years, 6 months ago (2016-06-16 19:12:52 UTC) #8
lijeffrey
comments addressed, ptal
4 years, 6 months ago (2016-06-16 19:13:07 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2056623002/20001
4 years, 6 months ago (2016-06-16 22:38:27 UTC) #12
commit-bot: I haz the power
Try jobs failed on following builders: Infra Presubmit on tryserver.infra (JOB_FAILED, https://build.chromium.org/p/tryserver.infra/builders/Infra%20Presubmit/builds/2415)
4 years, 6 months ago (2016-06-16 22:41:38 UTC) #14
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2056623002/40001
4 years, 6 months ago (2016-06-17 01:12:15 UTC) #17
commit-bot: I haz the power
4 years, 6 months ago (2016-06-17 01:39:29 UTC) #19
Message was sent while issue was closed.
Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/infra/infra/+/0539a5da9122baf2b74dd6ea7684d...

Powered by Google App Engine
This is Rietveld 408576698