-
Notifications
You must be signed in to change notification settings - Fork 150
[tracing] add OTEL quartz autoinstrumentation #7192
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 43 commits
Commits
Show all changes
102 commits
Select commit
Hold shift + click to select a range
1edb97f
hangfire sample app
chojomok 1cd2c26
add otel tracing to sample app
chojomok 1471305
send the otel traces to the agent
chojomok beb58a5
create Diagnostic Listener for Quartz
chojomok fafaa42
try duckstruct to populate args
chojomok 3513730
update span metdata
chojomok 7b0c45b
add additional span tags to match otel
chojomok 8835199
fix sample app
chojomok 732c136
setting up master quartz
chojomok 7e2bc8e
prep prerelease quartz
chojomok e21d895
creating quartz integration tests
chojomok 06616cf
add quartz snapshot
chojomok 8ece2ec
create versioned snapshots
chojomok 1c75b66
try to get packageversion as suffix
chojomok e881934
setup test to take variable packageversion (results in no package)
chojomok ae19e00
Update QuartzTestsV4.verified.txt
chojomok ab5f430
update version logic
chojomok ff412e7
generate proper package versions
chojomok 010a4ce
clean build after merge
chojomok 9b83d82
running generatepackage
chojomok 6800615
remove our custom instrumentation and add #nullable to listener
chojomok 838618e
Update PackageVersionsLatestMajors.g.props
chojomok 9041083
copy masters PackageVersionsLatestMinors.g.props
chojomok bb3274a
copy masters packageminor
chojomok f80de89
match generated code to master
chojomok 5de2a2b
generate package version changes just for quartz
chojomok 4c95501
remove , at the end of list (protobuf)
chojomok 583f2e4
Merge branch 'master' into mohamammad/quarz-integration
chojomok 4af1dd6
correct package name
chojomok 4c4e3d6
add new snapshot for 315
chojomok 03df157
Update QuartzTestsV315plus.verified.txt
chojomok 5366981
add snapshot for v3.15 netcorepp3.1
chojomok 1438a14
clean up
chojomok 7fc3620
Update tracer/test/Datadog.Trace.ClrProfiler.IntegrationTests/QuartzT…
chojomok 060b5d5
break up sample app
chojomok 53cc638
update isQuartz to include OTEL tags
chojomok c1583e4
remove component match
chojomok de33bd4
update v3 snapshot
chojomok cd3034e
update snapshots to match new file directory with sample app changes
chojomok 05d5042
Update tracer/test/test-applications/integrations/Samples.Quartz/Jobs…
chojomok 14a6b01
update integration tests
chojomok 57f2551
update launch settings
chojomok 336e710
update snapshots for v4 and added ordering to spans
chojomok 43d1c53
order by error on spans - should fail and give new snapshots
chojomok a8d18c8
add code for OTEL traces into sample app
chojomok 45fe58e
trying to create a ActivityHandler
chojomok aef38d8
Update QuartzActivityHandler.cs
chojomok c7ad15c
add logging to determine why the resource name is staying the same
chojomok 7302b12
adding veto jobs and activity handler updates
chojomok cf8fbb6
enhance resource name and operation name
chojomok 53e1bf9
add span.kind and exception to quartz activity + reflection for activ…
chojomok 1bff6a8
refactor span metadata enhance methods
chojomok 41e8ace
Merge branch 'master' into mohamammad/quarz-integration
chojomok ccec123
update v3 snapshot
chojomok a14c7bb
update v4 snapshot
chojomok 87cc7e6
remove updating span kind for quartz activity handler
chojomok f6fa8b0
remove quarz prerelease for nuget source
chojomok 01143b7
update tests to run on windows
chojomok 2b41d12
add try catch around CreateActivityKindSetter
chojomok 23efaea
update 3.1 core app snapshot + activity kind reflection
chojomok 9c9f067
Merge branch 'master' into mohamammad/quarz-integration
chojomok c410282
generalize function casting for setter
chojomok 88fffeb
try casting with int
chojomok cc7b50c
try IL to create setter method
chojomok 2e2422f
attempt at fixing System.ArgumentException
chojomok 988f440
fix types
chojomok c09f15c
remove IL for enum casting
chojomok f36f8ce
use activity.instance instead of ducktype class
chojomok 8ae7d55
remove empty string case for windows test
chojomok 7d01271
set default packageVersion
chojomok e422843
try GetData() to get package version
chojomok 958d6fd
skip test for .NETFramework
chojomok 49495e3
add back getdata
chojomok d94e3ef
Create QuartzTestsNETFRAMEWORK.verified.txt
chojomok 5cb92b2
skip test for .NETFramework
chojomok 065e91f
Merge branch 'master' into mohamammad/quarz-integration
chojomok aa4a9ad
Update tracer/src/Datadog.Trace/Activity/Handlers/QuartzActivityHandl…
chojomok 451817b
fix v4 snapshots and update operation name
chojomok 16baa62
null or empty is for windows, which should run the default of v3
chojomok 78d6cdf
update v4 snapshot to use newer preview version of 4.0
chojomok 7aece0b
fix scrubbing for v4 events
chojomok c0741e0
Merge branch 'master' into mohamammad/quarz-integration
chojomok d31be41
Merge branch 'master' into mohamammad/quarz-integration
chojomok 27e7b2e
Merge branch 'master' into mohamammad/quarz-integration
chojomok 8417d78
add debug logs to EnhanceActivityMetadata
chojomok 20aac07
comment out package references
chojomok 9844589
Update tracer/test/test-applications/integrations/Samples.Quartz/Prog…
chojomok bbf8a5f
Update tracer/build/PackageVersionsGeneratorDefinitions.json
chojomok f8efc23
Update tracer/src/Datadog.Trace/Activity/Handlers/QuartzActivityHandl…
chojomok eacf82c
Update tracer/src/Datadog.Trace/Activity/ActivityListener.cs
chojomok ff7fb1b
Update tracer/test/test-applications/integrations/Samples.Quartz/Prog…
chojomok a460bd4
add coreapp3.1 snapshots
chojomok b898efb
Merge branch 'master' into mohamammad/quarz-integration
chojomok feca4f3
Merge branch 'master' into mohamammad/quarz-integration
chojomok f290118
fixed typo for netcoreapp in quartz test
chojomok 25d549f
fix snapshot location
chojomok f5c76e0
update mockactivity5.displayname to have a setter
chojomok 775e67d
package version is null for windows, add check for netcoreapp
chojomok f0c729d
Merge branch 'master' into mohamammad/quarz-integration
chojomok 7c705c4
fix QuartzDiagnosticObserver.cs(58,25): error DDDUCK001
chojomok dc6b2e4
directives .netcoreapp3x instead of just 31
chojomok 94984e4
fix unreachable code
chojomok File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.