Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[AzureMonitorDistro] Remove logs sampling. #44745

Merged

Conversation

rajkumar-rangaraj
Copy link
Contributor

No description provided.

@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

Copy link
Contributor

@TimothyMothra TimothyMothra left a comment

Choose a reason for hiding this comment

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

LGTM. Left some minor comments.

One more thing; when we spoke offline you proposed moving the LogFilteringProcessor to the Demo project. Are you still planning on doing this?

@rajkumar-rangaraj rajkumar-rangaraj enabled auto-merge (squash) June 26, 2024 23:48
auto-merge was automatically disabled June 27, 2024 00:32

Pull request was closed

@rajkumar-rangaraj rajkumar-rangaraj enabled auto-merge (squash) June 27, 2024 00:32
@rajkumar-rangaraj rajkumar-rangaraj merged commit 6a9bff9 into Azure:main Jun 27, 2024
21 checks passed
tejasm-microsoft pushed a commit to tejasm-microsoft/azure-sdk-for-net that referenced this pull request Jul 22, 2024
* Remove logs sampling.

* changelog update

* pr feedback - changelog
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Monitor - Distro Monitor OpenTelemetry Distro
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants