New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 820129 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Mar 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

All informational builds failed at BuildPackages stage

Project Member Reported by x...@chromium.org, Mar 8 2018

Issue description

It started failing this morning, see an example builds:
https://uberchromegw.corp.google.com/i/chromeos.chrome/builders/peach_pit-tot-chrome-pfq-informational/builds/9908
See its error log: https://logs.chromium.org/v/?s=chromeos%2Fbb%2Fchromeos.chrome%2Fpeach_pit-tot-chrome-pfq-informational%2F9908%2F%2B%2Frecipes%2Fsteps%2FBuildPackages%2F0%2Fstdout

Selected error message:
chromeos-chrome-67.0.3366.0_alpha-r1: [247/43200] CXX obj/third_party/angle/translator/DeclareAndInitBuiltinsForInstancedMultiview.o
chromeos-chrome-67.0.3366.0_alpha-r1: [248/43200] ACTION //chrome/common/extensions/api:api_schema_generator(//build/toolchain/cros:target)
chromeos-chrome-67.0.3366.0_alpha-r1: [249/43200] CXX obj/crypto/crypto/nss_key_util.o
chromeos-chrome-67.0.3366.0_alpha-r1: FAILED: obj/crypto/crypto/nss_key_util.o 
chromeos-chrome-67.0.3366.0_alpha-r1: /home/chrome-bot/goma/gomacc armv7a-cros-linux-gnueabi-clang++ -B/usr/x86_64-pc-linux-gnu/armv7a-cros-linux-gnueabi/binutils-bin/2.27.0-gold -MMD -MF obj/crypto/crypto/nss_key_util.o.d -DCRYPTO_IMPLEMENTATION -DV8_DEPRECATION_WARNINGS -DUSE_UDEV -DUSE_AURA=1 -DUSE_NSS_CERTS=1 -DUSE_OZONE=1 -DFULL_SAFE_BROWSING -DSAFE_BROWSING_CSD -DSAFE_BROWSING_DB_LOCAL -DOFFICIAL_BUILD -DGOOGLE_CHROME_BUILD -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -DNO_UNWIND_TABLES -DCR_CLANG_REVISION=\"325667-1\" -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D_FORTIFY_SOURCE=2 -DOS_CHROMEOS -DNDEBUG -DNVALGRIND -DDYNAMIC_ANNOTATIONS_ENABLED=0 -I../../../../../../../home/chrome-bot/chrome_root/src -Igen -I../../../../../../../home/chrome-bot/chrome_root/src/third_party/boringssl/src/include -fno-strict-aliasing -fno-unwind-tables -fno-asynchronous-unwind-tables -fPIC -pipe -pthread -fcolor-diagnostics -no-canonical-prefixes --target=arm-linux-gnueabihf -march=armv7-a -mfloat-abi=hard -mtune=generic-armv7-a -mfpu=neon -mthumb -Wall -Wextra -Wimplicit-fallthrough -Wthread-safety -Wno-missing-field-initializers -Wno-unused-parameter -Wno-c++11-narrowing -Wno-covered-switch-default -Wno-unneeded-internal-declaration -Wno-inconsistent-missing-override -Wno-undefined-var-template -Wno-nonportable-include-path -Wno-address-of-packed-member -Wno-unused-lambda-capture -Wno-user-defined-warnings -Wno-enum-compare-switch -Wno-null-pointer-arithmetic -O2 -fno-ident -fdata-sections -ffunction-sections -fno-omit-frame-pointer -g2 -gsplit-dwarf -ggnu-pubnames -fvisibility=hidden -Wheader-hygiene -Wstring-conversion -Wtautological-overlap-compare -Wno-header-guard -isystem../../../../../../../build/peach_pit/build/peach_pit/usr/include/nss -isystem../../../../../../../build/peach_pit/build/peach_pit/usr/include/nspr -std=gnu++14 -fno-exceptions -fno-rtti --sysroot=../../../../../../../build/peach_pit -fvisibility-inlines-hidden -pipe -march=armv7-a -mtune=cortex-a15 -mfpu=neon -mfloat-abi=hard -fno-split-dwarf-inlining -fdebug-info-for-profiling -D__google_stl_debug_vector=1 -Wno-unknown-warning-option -stdlib=libc++  -c ../../../../../../../home/chrome-bot/chrome_root/src/crypto/nss_key_util.cc -o obj/crypto/crypto/nss_key_util.o
chromeos-chrome-67.0.3366.0_alpha-r1: In file included from ../../../../../../../home/chrome-bot/chrome_root/src/crypto/nss_key_util.cc:5:
chromeos-chrome-67.0.3366.0_alpha-r1: In file included from ../../../../../../../home/chrome-bot/chrome_root/src/crypto/nss_key_util.h:14:
chromeos-chrome-67.0.3366.0_alpha-r1: ../../../../../../../home/chrome-bot/chrome_root/src/crypto/scoped_nss_types.h:8:10: fatal error: 'keyhi.h' file not found
chromeos-chrome-67.0.3366.0_alpha-r1: #include <keyhi.h>
chromeos-chrome-67.0.3366.0_alpha-r1:          ^~~~~~~~~
chromeos-chrome-67.0.3366.0_alpha-r1: 1 error generated.

chromeos-chrome-67.0.3366.0_alpha-r1: ninja: build stopped: subcommand failed.
chromeos-chrome-67.0.3366.0_alpha-r1:  * ERROR: chromeos-base/chromeos-chrome-67.0.3366.0_alpha-r1::chromiumos failed (compile phase):
chromeos-chrome-67.0.3366.0_alpha-r1:  *   (no error message)

chromeos-chrome-67.0.3366.0_alpha-r1:  * CHROME_ORIGIN VALUE is LOCAL_SOURCE
chromeos-chrome-67.0.3366.0_alpha-r1:  * /home/chrome-bot/chrome_root should be set here properly
chromeos-chrome-67.0.3366.0_alpha-r1:  * ERROR: chromeos-base/chromeos-chrome-67.0.3366.0_alpha-r1::chromiumos failed (compile phase):
chromeos-chrome-67.0.3366.0_alpha-r1:  *   (no error message)


 

Comment 1 by x...@chromium.org, Mar 8 2018

Cc: tikuta@chromium.org
tikuta@, any possibility that it might be caused by your CL? 
https://chromium-review.googlesource.com/c/chromium/src/+/954526

Comment 2 by x...@chromium.org, Mar 8 2018

I'm going to revert this CL to see if it can fix the failure. 
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 8 2018

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

commit 7120df66904db2202bf6f69da975d02224307fd8
Author: Xiaoqian Dai <xdai@chromium.org>
Date: Thu Mar 08 18:12:18 2018

Revert "Use relative path for isysroot"

This reverts commit eebaaff05747257bfbc5d8613e571d7b4ca31b60.

Reason for revert: Suspect it broke the informational builds, see  crbug.com/820129  for details.

Bug:  820129 

Original change's description:
> Use relative path for isysroot
> 
> pkg_config added -isystem in abspath.
> 
> Let me use relative path here for cache utilization on goma backend when building chrome in different directory.
> 
> Change-Id: Id48db3036711c07788bb370753acc7a9b217a89d
> Reviewed-on: https://chromium-review.googlesource.com/954526
> Reviewed-by: Thomas Anderson <thomasanderson@chromium.org>
> Commit-Queue: Takuto Ikuta <tikuta@google.com>
> Cr-Commit-Position: refs/heads/master@{#541738}

TBR=tikuta@google.com,thomasanderson@chromium.org

Change-Id: I1eaf74f65f18b3c26607a230da320fae4b1f0f54
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/955677
Reviewed-by: Xiaoqian Dai <xdai@chromium.org>
Commit-Queue: Xiaoqian Dai <xdai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541838}
[modify] https://crrev.com/7120df66904db2202bf6f69da975d02224307fd8/build/config/linux/pkg_config.gni

Comment 4 by x...@chromium.org, Mar 8 2018

Status: Fixed (was: Untriaged)
Verified that this failure went away with the revert CL in place, see the latest build: https://uberchromegw.corp.google.com/i/chromeos.chrome/builders/peach_pit-tot-chrome-pfq-informational/builds/9927
Sorry for breakage, and thank you for reverting.

Sign in to add a comment