Skip to content

Commit

Permalink
ci: temproray disable vhds_integration_test in TSAN (envoyproxy#12067)
Browse files Browse the repository at this point in the history
- fix coverage number missed by envoyproxy#12054

Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: scheler <santosh.cheler@appdynamics.com>
  • Loading branch information
lizan authored and scheler committed Aug 4, 2020
1 parent 4a83db8 commit d360bec
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 1 deletion.
2 changes: 2 additions & 0 deletions .bazelrc
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,8 @@ build:clang-tsan --define ENVOY_CONFIG_TSAN=1
build:clang-tsan --copt -fsanitize=thread
build:clang-tsan --linkopt -fsanitize=thread
build:clang-tsan --linkopt -fuse-ld=lld
build:clang-tsan --build_tag_filters=-no_san,-no_tsan
build:clang-tsan --test_tag_filters=-no_san,-no_tsan
# Needed due to https://github.com/libevent/libevent/issues/777
build:clang-tsan --copt -DEVENT__DISABLE_DEBUG_MODE

Expand Down
6 changes: 5 additions & 1 deletion test/integration/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -217,7 +217,11 @@ envoy_cc_test(
data = [
"//test/config/integration/certs",
],
tags = ["fails_on_windows"],
tags = [
"fails_on_windows",
# https://github.com/envoyproxy/envoy/issues/9784
"no_tsan",
],
deps = [
":http_integration_lib",
"//source/common/config:protobuf_link_hacks",
Expand Down
1 change: 1 addition & 0 deletions test/per_file_coverage.sh
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ declare -a KNOWN_LOW_COVERAGE=(
"source/extensions/quic_listeners/quiche:84.8"
"source/extensions/retry:95.5"
"source/extensions/retry/host:85.7"
"source/extensions/retry/host/omit_canary_hosts:92.9"
"source/extensions/stat_sinks/statsd:85.2"
"source/extensions/tracers:96.5"
"source/extensions/tracers/opencensus:92.4"
Expand Down

0 comments on commit d360bec

Please sign in to comment.