-
Notifications
You must be signed in to change notification settings - Fork 153
[tracing] Create and reparent span for Azure API Management #7806
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
Open
jcstorms1
wants to merge
58
commits into
master
Choose a base branch
from
storms/add-azure-apim-proxy
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+869
−237
Open
Changes from all commits
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
5e25d91
initial
jcstorms1 1a133d3
remove unnecessary xml param
jcstorms1 8fbf66a
Add unit tests for extractors
jcstorms1 2e616ea
remove tracer paramater from integration tests
jcstorms1 fbaca0d
Fix unit tests
jcstorms1 1eb9c18
fix unit tests
jcstorms1 22c828b
add proxy when not aspnetcore
jcstorms1 f0e2ff5
Merge branch 'master' into storms/add-azure-apim-proxy
jcstorms1 80fdf63
Merge branch 'master' into storms/add-azure-apim-proxy
jcstorms1 548370a
working trace
jcstorms1 a9591b9
initial
jcstorms1 a400bd4
remove unnecessary xml param
jcstorms1 3d21dfe
Add unit tests for extractors
jcstorms1 32ed695
remove tracer paramater from integration tests
jcstorms1 301949c
Fix unit tests
jcstorms1 30152ca
fix unit tests
jcstorms1 8cdf247
add proxy when not aspnetcore
jcstorms1 b908490
working trace
jcstorms1 d55d815
Merge branch 'storms/add-azure-apim-proxy' of github.com:DataDog/dd-t…
jcstorms1 81aa696
removed all but 3 tags
jcstorms1 3285832
remove additional tags
jcstorms1 b97ec4c
Merge branch 'master' into storms/add-azure-apim-proxy
jcstorms1 21f1370
minor clean up
jcstorms1 204f0fa
additional tests
jcstorms1 6a2291d
remove csproj changes
jcstorms1 a587b44
spacing
jcstorms1 52a2b63
comments and nits
jcstorms1 6013731
restore azure functions csproj
jcstorms1 a8aefa2
add space
jcstorms1 5a26d82
readd space
jcstorms1 7d2b388
remove unused proxyname var
jcstorms1 c2c21cf
Merge branch 'master' into storms/add-azure-apim-proxy
jcstorms1 d657e14
add start time to proxy helper test
jcstorms1 7ae42ca
Merge branch 'storms/add-azure-apim-proxy' of github.com:DataDog/dd-t…
jcstorms1 36b2af9
remove unused import
jcstorms1 5a20d33
fix proxy span helper test
jcstorms1 c06d39f
fix azureproxyextractor unit tests
jcstorms1 f6aae2a
Merge branch 'master' into storms/add-azure-apim-proxy
jcstorms1 a301ddf
remove redundant tests
jcstorms1 a795862
refactor and fix timestamp issues
jcstorms1 79d390a
fix timestamps in inferredproxyspanhelpertests
jcstorms1 3d41bb7
Merge branch 'master' into storms/add-azure-apim-proxy
jcstorms1 8f54fb0
Merge branch 'master' into storms/add-azure-apim-proxy
jcstorms1 9b372a6
fix lost worker span and minor changes
jcstorms1 15d517e
Address initial comments
jcstorms1 7540a26
add and update tests
jcstorms1 9870aec
remove grpc struct
jcstorms1 707e511
update snapshots for span.kind change on proxy spans
jcstorms1 f6a7a67
Merge branch 'master' into storms/add-azure-apim-proxy
jcstorms1 3aae9b3
Merge branch 'master' into storms/add-azure-apim-proxy
jcstorms1 31c9a13
add function proxy integration test
jcstorms1 31f2a05
remove unecessary code block and changes
jcstorms1 2f10a13
Merge branch 'master' into storms/add-azure-apim-proxy
jcstorms1 6d000d2
remove more unnecessary code from worker process
jcstorms1 3715692
refactor and nullable references
jcstorms1 44cbe27
comments
jcstorms1 d87fd91
adding integration test and minor fixes
jcstorms1 c17569d
Merge branch 'master' into storms/add-azure-apim-proxy
jcstorms1 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
Some comments aren't visible on the classic Files Changed page.
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
25 changes: 0 additions & 25 deletions
25
...ace/ClrProfiler/AutoInstrumentation/Azure/Functions/Isolated/GrpcBindingsFeatureStruct.cs
This file was deleted.
Oops, something went wrong.
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.