Skip to content

Commit

Permalink
Merge pull request #4086 from DataDog/ivoanjo/re-enable-asan-testing
Browse files Browse the repository at this point in the history
[NO-TICKET] Re-enable ASAN memory leaks testing
  • Loading branch information
ivoanjo authored Nov 7, 2024
2 parents 27f7d0a + f1cd362 commit 2224fe4
Showing 1 changed file with 1 addition and 11 deletions.
12 changes: 1 addition & 11 deletions .github/workflows/test-memory-leaks.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,20 +14,10 @@ jobs:
- run: sudo apt-get update && (sudo apt-get install -y valgrind || sleep 5 && sudo apt-get install -y valgrind) && valgrind --version
- run: bundle exec rake compile spec:profiling:memcheck
test-asan:
# Temporarily disabled on 2024-09-17 until ruby-asan builds are available again on
# https://github.com/ruby/ruby-dev-builder/releases
if: false
runs-on: ubuntu-24.04
steps:
- uses: actions/checkout@v4
# We're using a fork of ruby/setup-ruby because the "asan" tool is built into the clang compiler toolchain, and
# needs Ruby to be built with a special configuration.
#
# The special configuration is not yet available in the upstream `ruby/setup-ruby` github action, so I needed to
# fork it and push a small tweak to make it available.
#
# (The Ruby builds were added in https://github.com/ruby/ruby-dev-builder/pull/10 ).
- uses: datadog/setup-ruby@0c7206d6db81faf999795ceebfac00d164298bd5
- uses: ruby/setup-ruby@v1
with:
ruby-version: asan
bundler-cache: true # runs 'bundle install' and caches installed gems automatically
Expand Down

0 comments on commit 2224fe4

Please sign in to comment.