Skip to content

ci: add TSAN integration test job - #1981

Open
nezdolik wants to merge 3 commits into
mainfrom
add-tsan-ci-check
Open

ci: add TSAN integration test job#1981
nezdolik wants to merge 3 commits into
mainfrom
add-tsan-ci-check

Conversation

@nezdolik

@nezdolik nezdolik commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Run Cilium proxy integration tests with ThreadSanitizer. TSan detects unsynchronized concurrent memory accesses and synchronization errors.

Run Cilium proxy integration tests with ThreadSanitizer.

Signed-off-by: Kateryna Nezdolii <kateryna.nezdolii@gmail.com>
@nezdolik
nezdolik requested a review from a team as a code owner August 6, 2026 13:41
@nezdolik
nezdolik requested a review from jrajahalme August 6, 2026 13:41
Defer closing the server sockets until the worker thread has exited.

Signed-off-by: Kateryna Nezdolii <kateryna.nezdolii@gmail.com>
@nezdolik
nezdolik force-pushed the add-tsan-ci-check branch from ae456aa to 0f00ff7 Compare August 7, 2026 12:02
@nezdolik

nezdolik commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

another failure in //tests:cilium_tls_tcp_integration_test https://github.com/cilium/proxy/actions/runs/31176562815/job/92859820354?pr=1981

Read fake-upstream TLS state on its dispatcher to avoid a TSAN race.
@nezdolik

Copy link
Copy Markdown
Contributor Author

all failures were only in test code

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