Repository navigation
test(async-profiler): [Sample Tests 33] Add an end-to-end test of a profiled transaction - #6325
Draft
runningcode wants to merge 1 commit into
Draft
runningcode wants to merge 1 commit into
runningcode wants to merge 1 commit into
Conversation
This was referenced Oct 20, 2026
Merged
Merged
Merged
Merged
Draft
Draft
5 of 9 tasks
runningcode
force-pushed
the
no/async-profiler-integration-test
branch
from
October 21, 2026 09:19
99dd98a to
b14e667
Compare
runningcode
force-pushed
the
no/core-integration-test
branch
from
October 21, 2026 09:19
23ddeb0 to
a75ec72
Compare
runningcode
force-pushed
the
no/async-profiler-integration-test
branch
from
October 21, 2026 11:27
b14e667 to
dfb3bba
Compare
runningcode
force-pushed
the
no/core-integration-test
branch
2 times, most recently
from
October 22, 2026 12:40
d8615b7 to
7053644
Compare
runningcode
force-pushed
the
no/async-profiler-integration-test
branch
2 times, most recently
from
October 22, 2026 13:31
641149e to
2d83690
Compare
runningcode
force-pushed
the
no/core-integration-test
branch
2 times, most recently
from
October 22, 2026 14:02
14bb678 to
8ace21b
Compare
runningcode
force-pushed
the
no/async-profiler-integration-test
branch
from
October 22, 2026 14:02
2d83690 to
332c74b
Compare
runningcode
force-pushed
the
no/core-integration-test
branch
from
October 22, 2026 16:02
8ace21b to
67c6c50
Compare
runningcode
force-pushed
the
no/async-profiler-integration-test
branch
from
October 22, 2026 16:02
332c74b to
83bca69
Compare
runningcode
force-pushed
the
no/core-integration-test
branch
from
October 23, 2026 08:24
67c6c50 to
a990e86
Compare
runningcode
force-pushed
the
no/async-profiler-integration-test
branch
from
October 23, 2026 08:24
83bca69 to
215aa1e
Compare
runningcode
force-pushed
the
no/core-integration-test
branch
from
October 23, 2026 08:58
a990e86 to
60de255
Compare
runningcode
force-pushed
the
no/async-profiler-integration-test
branch
from
October 23, 2026 08:58
215aa1e to
29b4976
Compare
runningcode
force-pushed
the
no/core-integration-test
branch
from
October 23, 2026 09:35
60de255 to
d5be157
Compare
runningcode
force-pushed
the
no/async-profiler-integration-test
branch
2 times, most recently
from
October 23, 2026 11:51
cac6e45 to
cec7d00
Compare
runningcode
force-pushed
the
no/core-integration-test
branch
from
October 23, 2026 11:51
d5be157 to
ace7cb1
Compare
runningcode
force-pushed
the
no/async-profiler-integration-test
branch
from
October 23, 2026 12:48
cec7d00 to
3b4b258
Compare
runningcode
force-pushed
the
no/core-integration-test
branch
2 times, most recently
from
October 23, 2026 13:02
1254e97 to
be7bcd3
Compare
runningcode
force-pushed
the
no/async-profiler-integration-test
branch
from
October 23, 2026 13:02
3b4b258 to
8539ee1
Compare
runningcode
force-pushed
the
no/core-integration-test
branch
from
October 23, 2026 13:53
be7bcd3 to
4264fa8
Compare
runningcode
force-pushed
the
no/async-profiler-integration-test
branch
from
October 23, 2026 13:53
8539ee1 to
26cd78b
Compare
ContinuousProfilerEndToEndTest starts a sampled transaction with the TRACE profile lifecycle and checks that its profile chunk reaches MockSentryServer, linked to the transaction by the profiler ID. It ports the profiling part of the console sample's system test. The profiler rejects a traces path with a dot anywhere in its absolute form, so the test puts it in the temp dir. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
runningcode
force-pushed
the
no/core-integration-test
branch
from
October 23, 2026 15:10
4264fa8 to
8ebabb9
Compare
runningcode
force-pushed
the
no/async-profiler-integration-test
branch
from
October 23, 2026 15:10
26cd78b to
2dd3597
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Stack (Sample Tests)
main)📜 Description
Adds
ContinuousProfilerEndToEndTesttosentry-async-profiler. It ports the profiling part ofsentry-samples-console's system test. A sampled transaction with theTRACEprofile lifecycle starts the async profiler, and the test checks that the profile chunk reachesMockSentryServerwith the transaction's profiler ID.The traces path is in the temp dir. The profiler rejects a traces path with a
.anywhere in its absolute form, as #6319 found, so a relative path fails in a checkout whose path has a dot in it.💡 Motivation and Context
sentry-async-profiler's tests mock the transport and the profiler's surroundings. This one runs the real profiler and sends the chunk over HTTP, in under a second.💚 How did you test it?
JavaContinuousProfilerTest's tests fail with or without this PR. The path contains/.claude/, and the same dot check disables profiling: one test expects,as the rejected character and gets.. In CI, whose checkout path has no dot, they pass.📝 Checklist
sendDefaultPIIis enabled.🔮 Next steps
The console sample's Kafka round trip into
sentry-kafka, which retires the sample.#skip-changelog
🤖 Generated with Claude Code