New issue
Advanced search Search tips

Issue 656573 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Oct 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Feature



Sign in to add a comment

Run valgrind leak checker on V8 linux64 release and trybot

Project Member Reported by machenb...@chromium.org, Oct 17 2016

Issue description

Tracker bug.
 
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 17 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/a276bc8998c0ec38ef9afbcb160e3b257c8afaa8

commit a276bc8998c0ec38ef9afbcb160e3b257c8afaa8
Author: machenbach <machenbach@chromium.org>
Date: Mon Oct 17 18:40:04 2016

[mb] Make linux64 bots aware of valgrind

Prepared by:
https://codereview.chromium.org/2423083002/

BUG= chromium:656573 
NOTRY=true
TBR=tandrii@chromium.org

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

[modify] https://crrev.com/a276bc8998c0ec38ef9afbcb160e3b257c8afaa8/infra/mb/mb_config.pyl

Project Member

Comment 3 by bugdroid1@chromium.org, Oct 17 2016

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

commit 829daf60e51ee9e6c78bea29918def500427ac76
Author: recipe-roller <recipe-roller@chromium.org>
Date: Mon Oct 17 22:31:12 2016

Roll recipe dependencies (trivial).

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/69fc06f12bb5fc602161bd2169cb54a03fe5634b Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/61728af22d26f33d09ed715d0c43f6051ee9c1ea Fix compile on the Blimp Client Engine Integration bot (shenghuazhang@chromium.org)
  https://crrev.com/f7b18bcd210f254d4b61c92b2a94942fe9b83927 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/dd839c91b9b2e7f1b45683dfe1e7ac68f01157a6 Modify 'build_dir' for run_mb() to avoid double 'Debug' (shenghuazhang@chromium.org)
  https://crrev.com/7eea10901de0274a5be22fdf6479e1109582900a Skia trigger recipe: change remote recipe condition (borenet@chromium.org)
  https://crrev.com/0b094b378658afa9436b1107859128456cd1c08d Add GomaCanaryiOS to chromium.fyi (smut@google.com)
  https://crrev.com/97367d4e07794d107d6075ff86ce898631169e73 Add option to use goma module in chromium.compile (tikuta@chromium.org)
  https://crrev.com/d44afde07794016523aceefdd1de603fc2f75459 V8: Run simple-leak-check step on release and trybot (machenbach@chromium.org)
  https://crrev.com/92dfa57e7376ea9ee60ce378cf16ae290cd9aea3 V8: Bump timeout on arm64 sim nosnap (machenbach@chromium.org)
  https://crrev.com/6b546c5afd0b236ef02e3dfb029d2d630582786c Revert of V8: Run simple-leak-check step on release and trybot (patchset #1 id:1 of https://codereview.chromium.org/2415243003/ ) (machenbach@chromium.org)
  https://crrev.com/94aa3062f0f4cc2542d375cf45706c7e2742c714 libyuv: Switch bots over to GN by default. (kjellander@chromium.org)
  https://crrev.com/ab47d254ddf187fa3b3c0231ea1882557b589f7b Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/3d9ee695a52c6ea7c00b60ae7ce8de03bbf626ed libyuv: Switch {client,tryserver}.libyuv to GN (kjellander@chromium.org)
  https://crrev.com/ded9582695d74387b613dcbcad04936bff12d88b libyuv: Disable signing for iOS (kjellander@chromium.org)
  https://crrev.com/1a9cedbf5751bb8b7d4a59ef43f9618b87b0b97f Updating flags necessary for upload_perf_dashboard_results.py (eyaich@chromium.org)
  https://crrev.com/1088f8337678d51a96fa44917bc9efc64270b0e4 Add the Syzygy x64 bots to the Syzygy waterfall. (sebmarchand@chromium.org)
  https://crrev.com/7ae96413206814a587e3e5921017762be6257983 Use the Chromium-side script to run the PGO benchmarks. (sebmarchand@chromium.org)
  https://crrev.com/02bbb17974a70e1bb198ee4243d830ff258d1bb8 Add IGNORE_DEPS_CHANGES flag and coverage builder. (dnj@chromium.org)
  https://crrev.com/78c3859a73773c4d7eff4ed38c796c147bae998c Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/c0b4818f62ae008fa858794ac278ce7f65cdf92d In archive_layout_test_results, only support archiving to GS. (qyearsley@chromium.org)
  https://crrev.com/65dc88015fad41b08d59d3f47ce157c3d5f8b051 V8: Add valgrind deps to linux64 bots (machenbach@chromium.org)
  https://crrev.com/9f623ff560eeca917efb9b799a1ca3137f9dcaa0 Revert of Add IGNORE_DEPS_CHANGES flag and coverage builder. (patchset #3 id:40001 of https://codereview.chromium.org/2418233002/ ) (dnj@chromium.org)
  https://crrev.com/124f02dd3006293789d17557aea29dfbb8f911e0 Remove use of "use_mirror" recipe property. (dnj@chromium.org)
  https://crrev.com/13c03fad59862d90b2b6bb2372a37ab414074228 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/2d442327bbfad0eb211466afbfe2f5a17c4561da Stop archiving redundant copy of {full,failing}_results.json. (qyearsley@chromium.org)
  https://crrev.com/506376486f9bd8973f5b96a02eddd449a3ef52e2 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
depot_tools:
  https://crrev.com/c9ebbd24639c8fb9f7fd2a36032614830e1e5285 Fix logic to detect Gerrit CL in CQ (rmistry@google.com)
  https://crrev.com/1dbb595dbfa4ebf73ee036cd66275af24253a99d Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/48df58136dc1847cd158eaa330f5dd457265d654 Gerrit git cl: make it work with minimal file. (tandrii@chromium.org)
  https://crrev.com/d437d76b64e083f2fb2ff3a7b38638d06642e270 Reland "Remove SVN and File support from gclient" (agable@chromium.org)
  https://crrev.com/aa9e16acd4e74f35dc738b98cd03b2c0b30bfefb Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/1c822ade1f731ec35a3d2d3be8805f773df22c4c Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
recipe_engine:
  https://crrev.com/a15c8474d308784e94fe4e15deb7c7860d717059 Reland: Add post_process hooks to recipe testing facilities. (iannucci@chromium.org)
  https://crrev.com/7076377b62701f77783d2d5e002fe2708ffab273 Fix VerifySubset when comparing empty lists (martiniss@chromium.org)
  https://crrev.com/bb6caf8060139fd4bcefee43f54ae00150e7ce3a Remove 'use_mirror' and 'root' properties. (dnj@chromium.org)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG= 656555 , chromium:656741 ,654806, chromium:656731 ,641588,chromium:633253, 656573 , 655810 , chromium:652188 , 655864 ,chromium:648308, chromium:656042 , 309849 

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2422383003
Cr-Commit-Position: refs/heads/master@{#425793}

[modify] https://crrev.com/829daf60e51ee9e6c78bea29918def500427ac76/infra/config/recipes.cfg

Project Member

Comment 4 by bugdroid1@chromium.org, Oct 18 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build.git/+/0cb76c4e79c13e8f742769452cc47a856373f352

commit 0cb76c4e79c13e8f742769452cc47a856373f352
Author: machenbach <machenbach@chromium.org>
Date: Tue Oct 18 06:26:33 2016

Reland V8: Run simple-leak-check step on release and trybot

Prepared by:
https://codereview.chromium.org/2423083002/
https://codereview.chromium.org/2427603002/

BUG= 656573 
TBR=tandrii@chromium.org

Committed: https://chromium.googlesource.com/chromium/tools/build/+/d44afde07794016523aceefdd1de603fc2f75459
Review-Url: https://codereview.chromium.org/2415243003

[modify] https://crrev.com/0cb76c4e79c13e8f742769452cc47a856373f352/scripts/slave/recipe_modules/v8/builders.py
[modify] https://crrev.com/0cb76c4e79c13e8f742769452cc47a856373f352/scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64.json
[modify] https://crrev.com/0cb76c4e79c13e8f742769452cc47a856373f352/scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64___builder.json
[modify] https://crrev.com/0cb76c4e79c13e8f742769452cc47a856373f352/scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64___builder_with_gn.json
[modify] https://crrev.com/0cb76c4e79c13e8f742769452cc47a856373f352/scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64_bisect_tester_swarming.json
[modify] https://crrev.com/0cb76c4e79c13e8f742769452cc47a856373f352/scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64_one_failure_build_env_not_supported.json
[modify] https://crrev.com/0cb76c4e79c13e8f742769452cc47a856373f352/scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux64_rel_ng.json
[modify] https://crrev.com/0cb76c4e79c13e8f742769452cc47a856373f352/scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux64_rel_ng_triggered.json

Project Member

Comment 5 by bugdroid1@chromium.org, Oct 18 2016

Project Member

Comment 6 by bugdroid1@chromium.org, Oct 18 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra.git/+/aede19073a37389a07729423498e5b53a121adbe

commit aede19073a37389a07729423498e5b53a121adbe
Author: recipe-roller <recipe-roller@chromium.org>
Date: Tue Oct 18 06:46:11 2016

Roll recipe dependencies (trivial).

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/0cb76c4e79c13e8f742769452cc47a856373f352 Reland V8: Run simple-leak-check step on release and trybot (machenbach@chromium.org)
  https://crrev.com/eec0e205b570e93edd338b5204dce218e8af4d77 libyuv: Remove all GYP builders. (kjellander@chromium.org)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG= 656573 , chromium:652188 

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2426733003

[modify] https://crrev.com/aede19073a37389a07729423498e5b53a121adbe/infra/config/recipes.cfg

Project Member

Comment 7 by bugdroid1@chromium.org, Oct 18 2016

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

commit f39e9d7188d82ba960a03317ee94d4388065d095
Author: recipe-roller <recipe-roller@chromium.org>
Date: Tue Oct 18 19:06:53 2016

Roll recipe dependencies (trivial).

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/21d67fdde732b8c961be656d6b6af4828970411a Revert of Use the Chromium-side script to run the PGO benchmarks. (patchset #1 id:40001 of https://codereview.chromium.org/2421363002/ ) (dimu@google.com)
  https://crrev.com/0cb76c4e79c13e8f742769452cc47a856373f352 Reland V8: Run simple-leak-check step on release and trybot (machenbach@chromium.org)
  https://crrev.com/eec0e205b570e93edd338b5204dce218e8af4d77 libyuv: Remove all GYP builders. (kjellander@chromium.org)
  https://crrev.com/4f91b12538a3360e4f70ed0ce9f4805d81ed7dc4 Replace linux_chromium_headless_dbg with linux_chromium_headless_rel (perezju@chromium.org)
  https://crrev.com/8063186c66ba88cfed50669b331771266edc863d libyuv: Set symbol_level=1 for Android builders. (kjellander@chromium.org)
  https://crrev.com/0aca4a6174120097e93b1f220a8c0c959862a084 libyuv: Make Windows trybots also build with VS2013. (kjellander@chromium.org)
  https://crrev.com/a76a98db1e63802b7c9569712d65ba2076cd8b73 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/cd408c7a6c68d0431f0a93982577cd198e757cc7 Skia Infra recipe: add DEPOT_TOOLS env variable for run_unittests (borenet@chromium.org)
  https://crrev.com/eb43e471e1a3d29858940c1469f5c7fb70960e2e Fixing arguments to perf upload script (eyaich@chromium.org)
depot_tools:
  https://crrev.com/69b1be10a44289cca2bbb1de51605ec28808dea0 Fix typo in --manually_grab_svn_rev deprecated flag. (sdefresne@chromium.org)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG=linux_chromium_headless_rel,656573,chromium:633253,656865,309849,skia:5723,chromium:652188

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://chromiumcodereview.appspot.com/2432473004
Cr-Commit-Position: refs/heads/master@{#426018}

[modify] https://crrev.com/f39e9d7188d82ba960a03317ee94d4388065d095/infra/config/recipes.cfg

Status: Verified (was: Started)

Sign in to add a comment