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

🚀 Feature: Milvus Instrumentation #244

Closed
1 task done
nirga opened this issue Dec 12, 2023 · 3 comments · Fixed by #1068
Closed
1 task done

🚀 Feature: Milvus Instrumentation #244

nirga opened this issue Dec 12, 2023 · 3 comments · Fixed by #1068
Labels
help wanted Extra attention is needed new instrumentation

Comments

@nirga
Copy link
Member

nirga commented Dec 12, 2023

Which component is this feature for?

All Packages

🔖 Feature description

Instrument calls to Milvus, including adding attributes, similarly to our Chroma instrumentation. The instrumentation should support all types of calls - streaming, non streaming, async, etc.

🎤 Why is this feature needed ?

Completness of OpenLLMetry

✌️ How do you aim to achieve this?

Similarily to other instrumentations we have in this repo.

🔄️ Additional Information

👀 Have you spent some time to check if this feature request has been raised before?

  • I checked and didn't find similar issue

Are you willing to submit PR?

None

@nirga nirga changed the title 🚀 Feature: Milvus Istrumentation 🚀 Feature: Milvus Instrumentation Dec 12, 2023
@nirga nirga added the help wanted Extra attention is needed label Jan 26, 2024
@Josecespedesant
Copy link
Contributor

Hello! Can I work on this issue?

@nirga
Copy link
Member Author

nirga commented May 4, 2024

Yes @Josecespedesant! Go ahead. Ping me on slack if you need assistance!

@Josecespedesant
Copy link
Contributor

Excellent, working on it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed new instrumentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants