Skip to content

Commit d460910

Browse files
committed
chore: Remove msan from all CIs except circleci.
It doesn't work yet and is wasting CI resources.
1 parent c5e3bca commit d460910

File tree

3 files changed

+17
-31
lines changed

3 files changed

+17
-31
lines changed

.cirrus.yml

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -50,23 +50,23 @@ bazel-asan_task:
5050
//c-toxcore/...
5151
-//c-toxcore/auto_tests:tcp_relay_test # TODO(robinlinden): Why does this pass locally but not in Cirrus?
5252

53-
# TODO(iphydf): Remove "|| true" once this works properly.
54-
bazel-msan_task:
55-
container:
56-
image: toxchat/toktok-stack:latest-msan
57-
cpu: 2
58-
memory: 4G
59-
configure_script:
60-
- /src/workspace/tools/inject-repo c-toxcore
61-
test_all_script:
62-
- cd /src/workspace && bazel test -k
63-
--remote_http_cache=http://$CIRRUS_HTTP_CACHE_HOST
64-
--build_tag_filters=-haskell
65-
--test_tag_filters=-haskell
66-
--remote_download_minimal
67-
--
68-
//c-toxcore/...
69-
-//c-toxcore/auto_tests:tcp_relay_test || true # TODO(robinlinden): Why does this pass locally but not in Cirrus?
53+
# TODO(iphydf): Enable once this works properly.
54+
#bazel-msan_task:
55+
# container:
56+
# image: toxchat/toktok-stack:latest-msan
57+
# cpu: 2
58+
# memory: 4G
59+
# configure_script:
60+
# - /src/workspace/tools/inject-repo c-toxcore
61+
# test_all_script:
62+
# - cd /src/workspace && bazel test -k
63+
# --remote_http_cache=http://$CIRRUS_HTTP_CACHE_HOST
64+
# --build_tag_filters=-haskell
65+
# --test_tag_filters=-haskell
66+
# --remote_download_minimal
67+
# --
68+
# //c-toxcore/...
69+
# -//c-toxcore/auto_tests:tcp_relay_test # TODO(robinlinden): Why does this pass locally but not in Cirrus?
7070

7171
# TODO(iphydf): Fix test timeouts.
7272
bazel-tsan_task:

.github/settings.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,13 +14,11 @@ branches:
1414
contexts:
1515
- "bazel-asan"
1616
- "bazel-dbg"
17-
- "bazel-msan"
1817
- "bazel-opt"
1918
- "bazel-tsan"
2019
- "bazel-valgrind"
2120
- "build-compcert"
2221
- "build-macos"
23-
- "build-msan"
2422
- "build-nacl"
2523
- "build-tcc"
2624
- "build-win32"
@@ -44,7 +42,6 @@ branches:
4442
- "docker-bootstrap-node-websocket"
4543
- "docker-toxcore-js"
4644
- "mypy"
47-
- "netlify/c-toxcore/deploy-preview"
4845
- "sonar-scan"
4946

5047
# Labels specific to c-toxcore.

.github/workflows/ci.yml

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -33,17 +33,6 @@ jobs:
3333
with:
3434
file: other/docker/tokstyle/Dockerfile
3535

36-
build-msan:
37-
runs-on: ubuntu-latest
38-
steps:
39-
- uses: actions/checkout@v2
40-
- name: Pull toxchat/toktok-stack:latest-msan
41-
run: docker pull toxchat/toktok-stack:latest-msan
42-
- name: Run tests under MemorySanitizer
43-
# TODO(iphydf): Remove "|| true" once this works correctly.
44-
run: docker run --rm -v $PWD:/src/workspace/c-toxcore toxchat/toktok-stack:latest-msan
45-
bazel test //c-toxcore/auto_tests:lossless_packet_test || true
46-
4736
build-nacl:
4837
runs-on: ubuntu-latest
4938
steps:

0 commit comments

Comments
 (0)