New issue
Advanced search Search tips

Issue 908140 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Dec 4
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

asan builder failing in UnitTest stage: authpolicy package failing to build

Project Member Reported by cmtice@google.com, Nov 23

Issue description

Starting around Nov. 21, the asan builder started failing in the UnitTest stage, failing to build authpolicy.

An example log can be found here.  Please fix this asap.

https://luci-logdog.appspot.com/logs/chromeos/buildbucket/cr-buildbucket.appspot.com/8929056462642132240/+/steps/UnitTest/0/stdout
 
Owner: ljusten@chromium.org
Status: Assigned (was: Untriaged)
is ljusten the correct owner? if not, please re-assign
He is. Taking a look.
Status: Started (was: Assigned)
This test checks whether seccomp filtering works, but minijail disables that for asan builds. The test just needs to be disabled for asan.

manojgupta@, is there some recommended way to disable specific tests when running under asan?
I had to disable a unit test for asan in libbrillo last year, you should be able to do something similar.
Those are the CLs:

https://chromium-review.googlesource.com/c/aosp/platform/external/libbrillo/+/898605
https://chromium-review.googlesource.com/c/aosp/platform/external/libbrillo/+/889659/

Project Member

Comment 6 by bugdroid1@chromium.org, Dec 3

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/c6c4fb2dde34c65968cf4a8f08bb82390ad5f118

commit c6c4fb2dde34c65968cf4a8f08bb82390ad5f118
Author: Lutz Justen <ljusten@chromium.org>
Date: Mon Dec 03 20:43:36 2018

authpolicy: Disable test under ASAN

Disable AuthPolicyTest.SeccompFiltersEnabled under ASAN. This test
checks whether seccomp filtering works, but minijail disables that for
ASAN builds, so the test fails.

BUG= chromium:908140 
TEST=Checked that test still runs when not under ASAN
     cros tryjob -g 1356740 betty-asan-tryjob

Change-Id: I969a051409672bc312212e869957d9b8d66cb04d
Reviewed-on: https://chromium-review.googlesource.com/1356740
Commit-Ready: Lutz Justen <ljusten@chromium.org>
Tested-by: Lutz Justen <ljusten@chromium.org>
Reviewed-by: Lutz Justen <ljusten@chromium.org>

[modify] https://crrev.com/c6c4fb2dde34c65968cf4a8f08bb82390ad5f118/authpolicy/authpolicy_unittest.cc

Owner: cmtice@google.com
Status: Fixed (was: Started)
cmtice@, could you please verify the fix?
is there any need for manual verification?
Status: Verified (was: Fixed)

Sign in to add a comment