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 OpenInference instrumentation packages to the registry #4609

Merged
merged 20 commits into from
Jun 7, 2024

Conversation

mikeldking
Copy link
Contributor

@mikeldking mikeldking commented Jun 6, 2024

This PR adds registry entries for the JS and Python instrumentation packages created and maintained by Arize AI's OSS team.

Copy link

linux-foundation-easycla bot commented Jun 6, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@mikeldking mikeldking force-pushed the openinference branch 2 times, most recently from 42301a6 to 576acd0 Compare June 6, 2024 06:16
cleanup

Update data/registry/instrumentation-js-openai.yml

Update data/registry/instrumentation-python-mistralai-openinference.yml

Update data/registry/instrumentation-python-openai-openinference.yml
@mikeldking mikeldking marked this pull request as ready for review June 6, 2024 06:39
@mikeldking mikeldking requested a review from a team June 6, 2024 06:39
@cartermp
Copy link
Contributor

cartermp commented Jun 6, 2024

/fix:all

@opentelemetrybot
Copy link
Collaborator

@mikeldking
Copy link
Contributor Author

@cartermp thanks so much for the review! cSpell is fixed.

Copy link
Contributor

@cartermp cartermp left a comment

Choose a reason for hiding this comment

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

Looks like the checker had a few more things it was unhappy with, heh. This should cover it.

data/registry/instrumentation-js-openai.yml Outdated Show resolved Hide resolved
data/registry/instrumentation-js-langchain.yml Outdated Show resolved Hide resolved
mikeldking and others added 3 commits June 6, 2024 18:15
Co-authored-by: Phillip Carter <pcarter@fastmail.com>
Co-authored-by: Phillip Carter <pcarter@fastmail.com>
Co-authored-by: Phillip Carter <pcarter@fastmail.com>
@mikeldking
Copy link
Contributor Author

@cartermp thanks again for the spelling review 🙏🏻

@svrnm svrnm merged commit 88a3629 into open-telemetry:main Jun 7, 2024
15 checks passed
@mikeldking mikeldking deleted the openinference branch June 7, 2024 15:16
ymotongpoo pushed a commit to ymotongpoo/opentelemetry.io that referenced this pull request Jun 8, 2024
…metry#4609)

Co-authored-by: Phillip Carter <pcarter@fastmail.com>
Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Severin Neumann <neumanns@cisco.com>
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.

5 participants