Skip to content

Add OpenLIT to Haystack Integrations #234

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

Merged
merged 3 commits into from
Jun 23, 2025
Merged

Conversation

patcher9
Copy link
Contributor

@patcher9 patcher9 commented May 22, 2024

How OpenLIT works with Haystack?

OpenLIT basically wraps around OpenAI, Anthropic etc functions and configures OpenTelemetry tracing and metrics for it. In the Haystack pipelines whenever the user runs a pipeline(Since Haystack also uses the same LLM functions), We are automatically able to wrap the pipeline (without needing to create any additional instrumentation for Haystack).

We did add instrumentation just for Document loaders as the rest of the pipeline can directly be tracked for reasons mentioned above.

@patcher9 patcher9 requested a review from a team as a code owner May 22, 2024 10:23
@patcher9
Copy link
Contributor Author

patcher9 commented Jul 9, 2024

@TuanaCelik , Tagging you for sanity as not sure who to tag for review on this :)

@TuanaCelik
Copy link
Contributor

Thanks for the contribution @patcher9 - Sorry we're slow at getting to PRs on this repository. I'm assigning this to myself now and will test it and make a review 🙏

@TuanaCelik TuanaCelik self-requested a review July 12, 2024 18:51
@bilgeyucel bilgeyucel requested review from bilgeyucel and removed request for TuanaCelik June 12, 2025 13:07
@bilgeyucel
Copy link
Contributor

Hi @patcher9, is this still up-to-date? If so, I can review it as soon as possible!

@patcher9
Copy link
Contributor Author

Hey @bilgeyucel It is very much up to date!

Update the code example
@bilgeyucel bilgeyucel merged commit 3af1338 into deepset-ai:main Jun 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants