Skip to content

feat(memtrack): add C++ and Rust integration tests - #212

Merged
not-matthias merged 12 commits into
mainfrom
cod-2002-add-rust-c-integration-tests-to-runner
Jan 30, 2026
Merged

feat(memtrack): add C++ and Rust integration tests#212
not-matthias merged 12 commits into
mainfrom
cod-2002-add-rust-c-integration-tests-to-runner

Conversation

@not-matthias

@not-matthias not-matthias commented Jan 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Member
  • Add C++ and Rust tests using common allocator functions
  • Also fixes a few issues discovered by the tests:
    • Adds support for previously missing posix_memalign and libstdc++ (libc but for C++)
    • Detect non-default allocators first for binaries
    • Attach to different jemalloc prefixed functions

@not-matthias
not-matthias force-pushed the cod-2002-add-rust-c-integration-tests-to-runner branch from 9477d9a to b3231ff Compare January 23, 2026 13:33
@codspeed-hq

codspeed-hq Bot commented Jan 23, 2026 •
edited
Loading

Copy link
Copy Markdown

CodSpeed Performance Report

Merging this PR will not alter performance

Comparing cod-2002-add-rust-c-integration-tests-to-runner (ca2c10c) with main (afa2c89)

Summary

✅ 4 untouched benchmarks

@not-matthias
not-matthias force-pushed the cod-2002-add-rust-c-integration-tests-to-runner branch 4 times, most recently from 63327c6 to cea4068 Compare January 23, 2026 16:31
@not-matthias
not-matthias force-pushed the cod-2002-add-rust-c-integration-tests-to-runner branch 3 times, most recently from 2670fb3 to 51536d2 Compare January 28, 2026 18:01
GuillaumeLagrange
GuillaumeLagrange approved these changes Jan 29, 2026
View reviewed changes

@GuillaumeLagrange GuillaumeLagrange left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

olgtm

Comment thread crates/memtrack/tests/shared.rs Outdated
Comment thread crates/memtrack/tests/c_tests.rs
Comment thread crates/memtrack/src/ebpf/memtrack.rs Outdated
@not-matthias
not-matthias force-pushed the cod-2002-add-rust-c-integration-tests-to-runner branch 5 times, most recently from 57cc5ef to f6a1e68 Compare January 30, 2026 13:41
@not-matthias
not-matthias force-pushed the cod-2002-add-rust-c-integration-tests-to-runner branch from f6a1e68 to ca2c10c Compare January 30, 2026 14:09
@not-matthias
not-matthias merged commit ca2c10c into main Jan 30, 2026
13 checks passed
@not-matthias
not-matthias deleted the cod-2002-add-rust-c-integration-tests-to-runner branch January 30, 2026 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@GuillaumeLagrange GuillaumeLagrange GuillaumeLagrange approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@not-matthias @GuillaumeLagrange