Skip to content

ci: add MSAN integration test job#1974

Merged
sayboras merged 1 commit into
mainfrom
add-msan-ci-check
Jul 17, 2026
Merged

ci: add MSAN integration test job#1974
sayboras merged 1 commit into
mainfrom
add-msan-ci-check

Conversation

@nezdolik

Copy link
Copy Markdown
Contributor

Running the existing C++ integration tests under MemorySanitizer adds coverage for use of uninitialized memory in exercised Cilium proxy code paths.

The MSAN job builds libcilium.so with Go sanitizer interoperability and the matching Clang toolchain so shadow state is preserved across the C++/Go boundary while retaining proxylib-backed tests.

Lets see if proxylib part works.

@nezdolik
nezdolik force-pushed the add-msan-ci-check branch 2 times, most recently from e5d52b5 to dac12be Compare July 15, 2026 12:07
@nezdolik
nezdolik marked this pull request as ready for review July 15, 2026 12:07
@nezdolik
nezdolik requested a review from a team as a code owner July 15, 2026 12:07
@nezdolik
nezdolik requested a review from sayboras July 15, 2026 12:07
@nezdolik
nezdolik force-pushed the add-msan-ci-check branch 3 times, most recently from ef2f6cc to bab13f1 Compare July 16, 2026 12:50
Running the existing C++ integration tests under MemorySanitizer adds coverage for use of uninitialized memory in exercised Cilium proxy code paths.

The MSAN job builds libcilium.so with Go sanitizer interoperability and the matching Clang toolchain so shadow state is preserved across the C++/Go boundary while retaining proxylib-backed tests. It reuses Dockerfile.tests with conservative parallelism, longer timeouts, a dedicated cache, and symbolized output.

Signed-off-by: Kateryna Nezdolii <kateryna.nezdolii@gmail.com>
@nezdolik
nezdolik force-pushed the add-msan-ci-check branch from bab13f1 to 857faf4 Compare July 17, 2026 11:15
@sayboras
sayboras merged commit a8397ac into main Jul 17, 2026
9 checks passed
@sayboras
sayboras deleted the add-msan-ci-check branch July 17, 2026 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants