New issue
Advanced search Search tips

Issue 622498 link

Starred by 0 users

Issue metadata

Status: Verified
Owner: ----
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

ChromeOS MSAN builds are busted in GN

Project Member Reported by dpranke@chromium.org, Jun 22 2016

Issue description

ChromeOS MSAN builders are busted in GN. See, e.g. 

https://build.chromium.org/p/chromium.memory.fyi/builders/Linux%20ChromeOS%20MSan%20Tests/builds/10674

It looks like we're probably misconfiguring or confusing the heck out of v8.

I'll flip the builder back to GYP for now.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 22 2016

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

commit 3eced20b7e61afd287a73551598ac2da42d864d1
Author: dpranke <dpranke@chromium.org>
Date: Wed Jun 22 22:55:49 2016

Flip "Linux ChromeOS MSan Builder" on memory.fyi back to GYP.

The GN build appears all kinds of busted.

TBR=reillyg@chromium.org, stevenjb@chromium.org
NOTRY=true
BUG= 622498 

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

[modify] https://crrev.com/3eced20b7e61afd287a73551598ac2da42d864d1/tools/mb/mb_config.pyl

Blocking: 621681
Labels: M-53
Not sure if this should block shipping, but I'll keep it on the radar for now.
Owner: ----
Status: Available (was: Assigned)
Labels: -M-53
likely won't make M-53 ...
Blocking: -621681
Labels: -Proj-GN-Migration Proj-GN-Migration-Ship
I'm going to try tracking ship-blocking things via the label "Proj-GN-Migration-Ship" (instead of Proj-GN-Migration) as suggested w/ a conversation w/ laforge@ and the monorail folks a while ago.

Querying for "Proj=GN-Migration%" should still return bugs for both labels, and this will allow us to filter out rollup things more easily, so that we can use blocking just for things that are truly blocking other bugs.
Blocking: -433082
Status: Fixed (was: Available)
This seems to be okay now.
Labels: VerifyIn-54
Status: Verified (was: Fixed)
bulk verified

Sign in to add a comment