Skip to content

Conversation

@fabistb
Copy link
Contributor

@fabistb fabistb commented Apr 20, 2025

Please follow this checklist before submitting:

  • Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • Read the contribution guide
  • Commands include options for Linux, MacOS, and Windows within codetabs
  • New file and folder names are globally unique
  • Page references use shortcodes instead of markdown or URL links
  • Images use HTML style and have alternative text
  • Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:

Description

Update the application insights otel collector application insights example to otel collector version 0.123.0

Issue reference

#4614

@fabistb fabistb marked this pull request as ready for review April 24, 2025 12:19
@fabistb fabistb requested review from a team as code owners April 24, 2025 12:19
@msfussell msfussell changed the title update app insigths otel example 0.123.0 Update Azure AppInsights otel example 0.123.0 Apr 25, 2025
@msfussell msfussell requested a review from alicejgibbons April 25, 2025 08:55
@msfussell msfussell added this to the 1.15 milestone Apr 25, 2025
@alicejgibbons
Copy link
Contributor

@fabistb I ended up testing your PR with App insights and figured we should use OTLP instead of Zipkin here since it is recommended. I migrated your changes to this PR: #4660. Mind closing this one and reviewing mine?

@fabistb
Copy link
Contributor Author

fabistb commented May 27, 2025

@fabistb I ended up testing your PR with App insights and figured we should use OTLP instead of Zipkin here since it is recommended. I migrated your changes to this PR: #4660. Mind closing this one and reviewing mine?

Great. I will close this PR and check #4660 .

@fabistb fabistb closed this May 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants