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.

Sign up for GitHub

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

Jump to bottom

Update vendors.yaml with APImetrics #4829

Closed
wants to merge 2 commits into from
Closed

Update vendors.yaml with APImetrics #4829

wants to merge 2 commits into from

Conversation

SophieMcK
Copy link
Contributor

No description provided.

@SophieMcK SophieMcK requested a review from a team July 12, 2024 16:49
@linux-foundation-easycla Linux Foundation: EasyCLA
Copy link

linux-foundation-easycla bot commented Jul 12, 2024 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

@tiffany76
Copy link
Contributor

Hi @SophieMcK, can you sign the CLA? We need your signature before we can merge this PR. Thanks!

@SophieMcK
Copy link
Contributor Author

Hi @tiffany76. CLA has been signed now

@svrnm
Copy link
Member

svrnm commented Jul 23, 2024

@SophieMcK can you provide us with some more details what your platform is doing? I read the following in your docs:

Send data from APImetrics to any platform that intakes OpenTelemetry with our OpenTelemetry Webhook

This sounds more like your platform sends data OUT in OpenTelemetry, and not that you take OpenTelemetry data IN?

@SophieMcK
Copy link
Contributor Author

Yes. That is the case

@svrnm
Copy link
Member

svrnm commented Jul 23, 2024

Yes. That is the case

Then I would argue that this might be an integration instead. I know that the lines between the different categories are blurry, but "vendors" are mostly backends (or observability pipelines) that consume opentelemetry data.

@SophieMcK
Copy link
Contributor Author

Okay. I'll close this and submit to the integrations page

@SophieMcK SophieMcK closed this Jul 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
CLA: no
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@SophieMcK @tiffany76 @svrnm