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

Add Integration for OpenLIT and a OpenTelemetry guide in Monitoring section #2593

Open
wants to merge 1 commit into
base: v2.4.x
Choose a base branch
from

Conversation

patcher9
Copy link

@patcher9 patcher9 commented May 24, 2024

This PR does two things

  • Adds OpenLIT to Integrations which talks about monitoring Milvus with OpenTelemetry (OpenLIT provides auto-instrumentation for monitoring Milvus operations via the SDK)
  • Creates a new page under Monitoring for OpenTelemetry and that doc has mostly similar content to what I added in the Integration doc

I would really appreciate some guidance on whats the best way to update the two docs

@sre-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: patcher9

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@patcher9 patcher9 changed the title Add Intgeration for OpenLIT and a OpenTelemetry guide in Monitoring section Add Integration for OpenLIT and a OpenTelemetry guide in Monitoring section May 24, 2024
@patcher9
Copy link
Author

/auto-cc

@patcher9
Copy link
Author

patcher9 commented Dec 9, 2024

tagging @liyun95 / @AnthonyTsu1984 as I am not sure who to tag for a review, Thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants