-
Notifications
You must be signed in to change notification settings - Fork 4.5k
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
[AzureMonitorDistro] upgrade OpenTelemetry for hotfix #43432
[AzureMonitorDistro] upgrade OpenTelemetry for hotfix #43432
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Central package versions look good.
API change check API changes are not detected in this pull request. |
sdk/monitor/Azure.Monitor.OpenTelemetry.AspNetCore/CHANGELOG.md
Outdated
Show resolved
Hide resolved
Updated the changelog with the quote from the OTel changelog. |
sdk/monitor/Azure.Monitor.OpenTelemetry.AspNetCore/CHANGELOG.md
Outdated
Show resolved
Hide resolved
sdk/monitor/Azure.Monitor.OpenTelemetry.AspNetCore/CHANGELOG.md
Outdated
Show resolved
Hide resolved
sdk/monitor/Azure.Monitor.OpenTelemetry.AspNetCore/CHANGELOG.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Rajkumar Rangaraj <rajrang@microsoft.com>
sdk/monitor/Azure.Monitor.OpenTelemetry.AspNetCore/CHANGELOG.md
Outdated
Show resolved
Hide resolved
sdk/monitor/Azure.Monitor.OpenTelemetry.AspNetCore/CHANGELOG.md
Outdated
Show resolved
Hide resolved
sdk/monitor/Azure.Monitor.OpenTelemetry.AspNetCore/CHANGELOG.md
Outdated
Show resolved
Hide resolved
I suggest we overwrite the default redaction in the instrumentation library and continue with the same behavior UNTIL OpenTelemetry makes a spec change. This minimizes the risk of giving our customers whip lash on a behavior that has been standard in Application Insight for most/all of its existence and ensures we have a consistent product position across languages. |
+1 on keeping the existing behavior for now |
Changing this to Draft. Need to make some changes. Will aim to get this merged by tomorrow (Fri) morning. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
sdk/monitor/Azure.Monitor.OpenTelemetry.AspNetCore/CHANGELOG.md
Outdated
Show resolved
Hide resolved
f5b43e5
into
hotfix/azure.monitor.opentelemetry.aspnetcore_1.1.1
Thank you! |
* [AzureMonitorDistro] upgrade OpenTelemetry for hotfix (#43432) * hotfix * update changelog * update other packages as well * Update sdk/monitor/Azure.Monitor.OpenTelemetry.AspNetCore/CHANGELOG.md Co-authored-by: Rajkumar Rangaraj <rajrang@microsoft.com> * update changelog with feedback * fix link * add link to breaking changes section * change Distro to override Disable * update changelog * update changelog * update changelog * typo * update changelog --------- Co-authored-by: Rajkumar Rangaraj <rajrang@microsoft.com> * fix broken link * fix broken link * update changelog * edit changelog --------- Co-authored-by: Rajkumar Rangaraj <rajrang@microsoft.com>
I created this PR so our team can review these changes.
Hotfix
I created a new
hotfix/azure.monitor.opentelemetry.aspnetcore_1.1.1
, forked from tag/Azure.Monitor.OpenTelemetry.AspNetCore_1.1.0Our hotfix will release from this branch.
This PR is merging into this branch.
Changes
This PR is upgrades the OpenTelemetry dependencies to 1.8.1.