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

Issue 2055623002: lkgr master: Port linux msan, tsan and windows asan bots to recipes (Closed)

Created:
4 years, 6 months ago by Nico
Modified:
4 years, 6 months ago
Reviewers:
Dirk Pranke, krasin1
CC:
chromium-reviews, infra-reviews+build_chromium.org, kjellander-cc_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/tools/build@master
Target Ref:
refs/heads/master
Project:
build
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : rebase #

Patch Set 3 : . #

Total comments: 1

Patch Set 4 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+364 lines, -297 lines) Patch
M masters/master.chromium.lkgr/master_lkgr_cfg.py View 1 2 3 3 chunks +18 lines, -108 lines 0 comments Download
M scripts/slave/recipe_modules/chromium/config.py View 3 chunks +9 lines, -3 lines 0 comments Download
M scripts/slave/recipe_modules/chromium_tests/chromium_lkgr.py View 2 chunks +127 lines, -0 lines 0 comments Download
A + scripts/slave/recipes/chromium.expected/full_chromium_lkgr_MSAN_Release__chained_origins_.json View 6 chunks +9 lines, -9 lines 0 comments Download
A + scripts/slave/recipes/chromium.expected/full_chromium_lkgr_MSAN_Release__no_origins_.json View 6 chunks +9 lines, -9 lines 0 comments Download
A + scripts/slave/recipes/chromium.expected/full_chromium_lkgr_TSAN_Debug.json View 9 chunks +14 lines, -14 lines 0 comments Download
A + scripts/slave/recipes/chromium.expected/full_chromium_lkgr_TSAN_Release.json View 6 chunks +9 lines, -9 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_lkgr_UBSan_vptr_Release.json View 1 1 chunk +1 line, -1 line 0 comments Download
A + scripts/slave/recipes/chromium.expected/full_chromium_lkgr_Win_ASan_Release.json View 1 17 chunks +56 lines, -48 lines 0 comments Download
A + scripts/slave/recipes/chromium.expected/full_chromium_lkgr_Win_ASan_Release_Coverage.json View 17 chunks +56 lines, -48 lines 0 comments Download
A + scripts/slave/recipes/chromium.expected/full_chromium_lkgr_Win_ASan_Release_Media.json View 17 chunks +56 lines, -48 lines 0 comments Download

Messages

Total messages: 17 (8 generated)
Nico
4 years, 6 months ago (2016-06-08 22:15:19 UTC) #3
Dirk Pranke
lgtm, thanks also! Hopefully you double-checked the GYP_DEFINES? The rest of the CL is straightforward. ...
4 years, 6 months ago (2016-06-08 22:21:53 UTC) #4
Nico
On 2016/06/08 22:21:53, Dirk Pranke wrote: > lgtm, thanks also! > > Hopefully you double-checked ...
4 years, 6 months ago (2016-06-09 00:18:39 UTC) #5
Nico
https://codereview.chromium.org/2055623002/diff/40001/scripts/slave/recipe_modules/chromium_tests/chromium_lkgr.py File scripts/slave/recipe_modules/chromium_tests/chromium_lkgr.py (right): https://codereview.chromium.org/2055623002/diff/40001/scripts/slave/recipe_modules/chromium_tests/chromium_lkgr.py#newcode100 scripts/slave/recipe_modules/chromium_tests/chromium_lkgr.py:100: 'chromium_apply_config': ['mb', 'tsan2', 'clobber'], (this is missing debug_extra_cflags="-gline-tables-only" in ...
4 years, 6 months ago (2016-06-09 00:24:11 UTC) #6
Nico
Finished auditing. Another difference is that this enables goma on the win asan bots, which ...
4 years, 6 months ago (2016-06-09 00:32:48 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2055623002/40001
4 years, 6 months ago (2016-06-09 00:32:59 UTC) #10
commit-bot: I haz the power
Try jobs failed on following builders: Build Presubmit on tryserver.infra (JOB_FAILED, https://build.chromium.org/p/tryserver.infra/builders/Build%20Presubmit/builds/5330)
4 years, 6 months ago (2016-06-09 00:41:10 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2055623002/60001
4 years, 6 months ago (2016-06-09 00:42:39 UTC) #15
commit-bot: I haz the power
4 years, 6 months ago (2016-06-09 00:47:00 UTC) #17
Message was sent while issue was closed.
Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/tools/build/+/0d230b31eb7052d502b7...

Powered by Google App Engine
This is Rietveld 408576698