Skip to content
This repository was archived by the owner on Sep 17, 2025. It is now read-only.

Fix HTTP attributes for deps, fix success and response/resultCode for Azure Exporter#827

Merged
lzchen merged 20 commits intocensus-instrumentation:masterfrom
lzchen:azure
Jan 28, 2020
Merged

Fix HTTP attributes for deps, fix success and response/resultCode for Azure Exporter#827
lzchen merged 20 commits intocensus-instrumentation:masterfrom
lzchen:azure

Conversation

@lzchen
Copy link
Copy Markdown
Contributor

@lzchen lzchen commented Nov 25, 2019

Address [#825] and [#826], [#831] and [#832]

Adds "component" span attributes to requests and httplib integrations to specify "http" type.
Adds logic to popular success and responseCode based off span status or status_code in span attributes.

@lzchen lzchen requested review from a team, c24t, reyang and songy23 as code owners November 25, 2019 19:54
@lzchen
Copy link
Copy Markdown
Contributor Author

lzchen commented Dec 2, 2019

@lmolkova
Please review if you have the time. It consists of the same changes as for OpenTelemetry.

@lzchen lzchen changed the title Fix HTTP attributes on Azure Exporter for dependencies Fix HTTP attributes for deps, fix success and response/resultCode for Azure Exporter Dec 31, 2019
@lzchen
Copy link
Copy Markdown
Contributor Author

lzchen commented Jan 21, 2020

@lmolkova
I have made the changes you've suggested!

Copy link
Copy Markdown
Member

@c24t c24t left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lzchen lzchen merged commit 3b48294 into census-instrumentation:master Jan 28, 2020
@lzchen lzchen deleted the azure branch January 28, 2020 21:29
@lzchen lzchen mentioned this pull request Feb 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants