Skip to content
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

[chore][exporter/azuremonitor] Force the telemetry to be sent to azuremonitor #36520

Merged
merged 4 commits into from
Nov 25, 2024

Conversation

whitneygriffith
Copy link
Contributor

Description

  1. Resolved an issue where traces weren't being sent to App Insights due to not flushing the Telemetry Channel. Added the necessary flush operation to ensure all traces, metrics and logs are properly sent to the queue, leveraging App Insights' batch handling for more efficient processing.

Link to tracking issue

Resolves #35037

Copy link
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

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

@codeboten codeboten merged commit 62998b5 into open-telemetry:main Nov 25, 2024
161 checks passed
@github-actions github-actions bot added this to the next release milestone Nov 25, 2024
@whitneygriffith whitneygriffith deleted the azuremonitor-bug-fix branch November 25, 2024 20:09
shivanthzen pushed a commit to shivanthzen/opentelemetry-collector-contrib that referenced this pull request Dec 5, 2024
…emonitor (open-telemetry#36520)

#### Description

1. Resolved an issue where traces weren't being sent to App Insights due
to not flushing the Telemetry Channel. Added the necessary flush
operation to ensure all traces, metrics and logs are properly sent to
the queue, leveraging App Insights' batch handling for more efficient
processing.

#### Link to tracking issue
Resolves open-telemetry#35037

---------

Signed-off-by: whitneygriffith <[email protected]>
Co-authored-by: Andrzej Stencel <[email protected]>
ZenoCC-Peng pushed a commit to ZenoCC-Peng/opentelemetry-collector-contrib that referenced this pull request Dec 6, 2024
…emonitor (open-telemetry#36520)

#### Description

1. Resolved an issue where traces weren't being sent to App Insights due
to not flushing the Telemetry Channel. Added the necessary flush
operation to ensure all traces, metrics and logs are properly sent to
the queue, leveraging App Insights' batch handling for more efficient
processing.

#### Link to tracking issue
Resolves open-telemetry#35037

---------

Signed-off-by: whitneygriffith <[email protected]>
Co-authored-by: Andrzej Stencel <[email protected]>
sbylica-splunk pushed a commit to sbylica-splunk/opentelemetry-collector-contrib that referenced this pull request Dec 17, 2024
…emonitor (open-telemetry#36520)

#### Description

1. Resolved an issue where traces weren't being sent to App Insights due
to not flushing the Telemetry Channel. Added the necessary flush
operation to ensure all traces, metrics and logs are properly sent to
the queue, leveraging App Insights' batch handling for more efficient
processing.

#### Link to tracking issue
Resolves open-telemetry#35037

---------

Signed-off-by: whitneygriffith <[email protected]>
Co-authored-by: Andrzej Stencel <[email protected]>
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.

Istio Traces Aren't Being Exported with Azure Monitor Exporter
4 participants