New issue
Advanced search Search tips

Issue 666820 link

Starred by 1 user

Issue metadata

Status: Duplicate
Owner: ----
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

"CTPolicyEnforcerTest.ConformsToCTEVPolicyIfSCTBeforeEnforcementDate" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Nov 18 2016

Issue description

"CTPolicyEnforcerTest.ConformsToCTEVPolicyIfSCTBeforeEnforcementDate" is flaky.

This issue was created automatically by the chromium-try-flakes app. Please find the right owner to fix the respective test/step and assign this issue to them. If the step/test is infrastructure-related, please add Infra-Troopers label and change issue status to Untriaged. When done, please remove the issue from Sheriff Bug Queue by removing the Sheriff-Chromium label.

We have detected 5 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyTgsSBUZsYWtlIkNDVFBvbGljeUVuZm9yY2VyVGVzdC5Db25mb3Jtc1RvQ1RFVlBvbGljeUlmU0NUQmVmb3JlRW5mb3JjZW1lbnREYXRlDA.

Flaky tests should be disabled within 30 minutes unless culprit CL is found and reverted. Please see more details here: https://sites.google.com/a/chromium.org/dev/developers/tree-sheriffs/sheriffing-bug-queues#triaging-auto-filed-flakiness-bugs
 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 18 2016

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

commit afb4d4b1e2792de2632e16d44616a8bcfe0f5a86
Author: fsamuel <fsamuel@chromium.org>
Date: Fri Nov 18 21:47:28 2016

Disable CTPolicyEnforcerTest.ConformsToCTEVPolicyIfSCTBeforeEnforcementDate

It's flaky on Android

BUG= 666820 
TBR=estade@chromium.org
NOTRY=true

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

[modify] https://crrev.com/afb4d4b1e2792de2632e16d44616a8bcfe0f5a86/net/cert/ct_policy_enforcer_unittest.cc

Project Member

Comment 2 by bugdroid1@chromium.org, Nov 18 2016

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

commit cefd44b8edf187c655024c9b5eabb166422bcd2c
Author: mdjones <mdjones@chromium.org>
Date: Fri Nov 18 22:04:17 2016

Revert of Disable CTPolicyEnforcerTest.ConformsToCTEVPolicyIfSCTBeforeEnforcementDate (patchset #1 id:1 of https://codereview.chromium.org/2517673003/ )

Reason for revert:
Causes compile failure:

FAILED: obj/net/_net_unittests__library/ct_policy_enforcer_unittest.o
/mnt/data/b/build/slave/cache/cipd/goma/gomacc ../../third_party/android_tools/ndk/toolchains/aarch64-linux-android-4.9/prebuilt/linux-x86_64/bin/aarch64-linux-android-g++ -MMD -MF obj/net/_net_unittests__library/ct_policy_enforcer_unittest.o.d -DUSE_KERBEROS -DV8_DEPRECATION_WARNINGS -DENABLE_NOTIFICATIONS -DUSE_OPENSSL_CERTS=1 -DNO_TCMALLOC -DUSE_EXTERNAL_POPUP_MENU=1 -DENABLE_WEBRTC=1 -DDISABLE_NACL -DUSE_PROPRIETARY_CODECS -DSAFE_BROWSING_DB_REMOTE -DCHROMIUM_BUILD -DENABLE_MEDIA_ROUTER=1 -DENABLE_WEBVR -DFIELDTRIAL_TESTING_ENABLED -D_FILE_OFFSET_BITS=64 -DANDROID -DHAVE_SYS_UIO_H -DANDROID_NDK_VERSION=r12b -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__GNU_SOURCE=1 -D_DEBUG -DDYNAMIC_ANNOTATIONS_ENABLED=1 -DWTF_USE_DYNAMIC_ANNOTATIONS=1 -DGTEST_HAS_POSIX_RE=0 -DGTEST_LANG_CXX11=1 -DGOOGLE_PROTOBUF_NO_RTTI -DGOOGLE_PROTOBUF_NO_STATIC_INITIALIZER -DHAVE_PTHREAD -DU_USING_ICU_NAMESPACE=0 -DU_ENABLE_DYLOAD=0 -DU_STATIC_IMPLEMENTATION -DICU_UTIL_DATA_IMPL=ICU_UTIL_DATA_FILE -DUNIT_TEST -I../.. -Igen -I../../testing/gtest/include -I../../third_party/protobuf/src -Igen/protoc_out -I../../third_party/protobuf/src -I../../third_party/boringssl/src/include -I../../third_party/ced/src -I../../third_party/icu/source/common -I../../third_party/icu/source/i18n -I../../testing/gmock_custom -I../../testing/gmock/include -I../../third_party/zlib -I../../v8/include -Igen/v8/include -Igen/net/net_test_jni_headers -Igen/net/net_test_jni_headers/net/test -fno-strict-aliasing --param=ssp-buffer-size=4 -fstack-protector -funwind-tables -fPIC -pipe -ffunction-sections -fno-short-enums -finline-limit=64 -Wall -Werror -Wno-unused-local-typedefs -Wno-maybe-uninitialized -Wno-missing-field-initializers -Wno-unused-parameter -Os -fno-omit-frame-pointer -gdwarf-3 -g1 --sysroot=../../third_party/android_tools/ndk/platforms/android-21/arch-arm64 -fvisibility=hidden -fno-threadsafe-statics -fvisibility-inlines-hidden -std=gnu++11 -Wno-narrowing -fno-rtti -isystem../../third_party/android_tools/ndk/sources/cxx-stl/llvm-libc++/libcxx/include -isystem../../third_party/android_tools/ndk/sources/cxx-stl/llvm-libc++abi/libcxxabi/include -isystem../../third_party/android_tools/ndk/sources/android/support/include -fno-exceptions -c ../../net/cert/ct_policy_enforcer_unittest.cc -o obj/net/_net_unittests__library/ct_policy_enforcer_unittest.o
../../net/cert/ct_policy_enforcer_unittest.cc:183:1: error: 'DISABLED_ConformsToCTEVPolicyIfSCTBeforeEnforcementDate' does not name a type
 DISABLED_ConformsToCTEVPolicyIfSCTBeforeEnforcementDate

Original issue's description:
> Disable CTPolicyEnforcerTest.ConformsToCTEVPolicyIfSCTBeforeEnforcementDate
>
> It's flaky on Android
>
> BUG= 666820 
> TBR=estade@chromium.org
> NOTRY=true
>
> Committed: https://crrev.com/afb4d4b1e2792de2632e16d44616a8bcfe0f5a86
> Cr-Commit-Position: refs/heads/master@{#433307}

TBR=fsamuel@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 666820 

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

[modify] https://crrev.com/cefd44b8edf187c655024c9b5eabb166422bcd2c/net/cert/ct_policy_enforcer_unittest.cc

Cc: jbudorick@chromium.org
This is either  Issue 665686  or an issue with the bot. From briefly scanning the test failures, they all looked to be date-sensitive tests.
Mergedinto: 666821
Status: Duplicate (was: Untriaged)

Sign in to add a comment