Skip to content

Commit

Permalink
[NO-TICKET] Re-enable memory leak asan testing in CI
Browse files Browse the repository at this point in the history
**What does this PR do?**

This PR reverts #3915 where we temporarily disable memory leak
testing using the asan tool because upstream builds were broken.

Additionally, since ruby/setup-ruby#653
the setup-ruby action now fully supports the asan builds, so we no
longer require our fork.

**Motivation:**

The asan tool does quite extensive checks in the profiler test
suite, so having it running in CI helps catch issues that may
otherwise slip by.

**Additional Notes:**

N/A

**How to test the change?**

Validate that the "Test for memory leaks -> test-asan" CI job is
running and passing successfully.
  • Loading branch information
ivoanjo committed Oct 7, 2024
1 parent 2cde72a commit f158428
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 f158428

Please sign in to comment.