chromeos-base/minijail unittest failing |
||||
Issue descriptionExamples: lakitu: https://uberchromegw.corp.google.com/i/chromiumos.tryserver/builders/llvm_next_toolchain/builds/1243 link: https://uberchromegw.corp.google.com/i/chromiumos.tryserver/builders/llvm_next_toolchain/builds/1244 The log contains: make[1]: *** [common.mk:526: TEST(CXX_BINARY(minijail0_cli_unittest))] Error 1 Only observed on the llvm-next rotating builders so far, so suspect this has something to do with new version of llvm.
,
Feb 2 2018
Was it a full build with llvm-next? Maybe the root cause is one of the minijail dependencies. Can you try: $ emerge-lakitu -e chromeos-base/minijail -j32 // Rebuild all dependent packages $ FEATURES=test emerge-lakitu chromeos-base/minijail
,
Feb 2 2018
It was not full build. I'll try again with full build. However, lakitu unittests have passed before with llvm-next: https://uberchromegw.corp.google.com/i/chromiumos.tryserver/builders/llvm_next_toolchain/builds/1194 So if this is specific to llvm-next, it was probably triggered by some other change in minijail or its dependencies after Jan 27.
,
Feb 2 2018
Yes, there is a chance that some dependent package is misbehaving. IIRC, in a recent llvm-next roll, one of the UnitTest fails was in Imageloader that was actually caused by dev-libs/nss miscompilation. If llvm-next full repros the problem, then package bisection should help.
,
Feb 5 2018
,
Feb 6 2018
,
Feb 6 2018
that sounds like the bug! THanks Mike!
,
Feb 6 2018
Thanks Mike. I applied the patch locally and confirmed that it fixes the issue.
,
Feb 8 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/35114d479e6f0a88a7f22df8c9528f7dc511ab83 commit 35114d479e6f0a88a7f22df8c9528f7dc511ab83 Author: Mike Frysinger <vapier@chromium.org> Date: Thu Feb 08 09:24:08 2018 minijail: pull in unittest fix 9741372f23e7 minijail0_cli_unittest: fix random crashes/failures BUG= chromium:808559 TEST=precq passes Change-Id: I9f56ff8b8e624bbe6635edfc1274a78d16e97240 Reviewed-on: https://chromium-review.googlesource.com/907073 Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com> Tested-by: Mike Frysinger <vapier@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org> [rename] https://crrev.com/35114d479e6f0a88a7f22df8c9528f7dc511ab83/chromeos-base/minijail/minijail-1-r6.ebuild
,
Feb 8 2018
,
Jun 21 2018
|
||||
►
Sign in to add a comment |
||||
Comment 1 by rahulchaudhry@chromium.org
, Feb 2 2018