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

Make buildevents more generic by using OpenTelemetry instead of beeline lib. #125

Open
ringods opened this issue Nov 4, 2021 · 3 comments
Labels
type: enhancement New feature or request

Comments

@ringods
Copy link

ringods commented Nov 4, 2021

Is your feature request related to a problem? Please describe.

No.

Describe the solution you'd like

The buildevents library is very handy to get insight into a CI system, but the current implementation is limited to Honeycomb.io because it builds against the Honeycomb beeline library. If we would switch to use the Opentelemetry Go SDK, output could be collected on anything supporting the OpenTelemetry Collector, still including Honeycomb.io.

Describe alternatives you've considered

Searched for alternatives, but haven't found anything which came as close to buildevents

Additional context

@ringods ringods added the type: enhancement New feature or request label Nov 4, 2021
@MikeGoldsmith
Copy link
Contributor

Hey @ringods - thanks for the feature request. We'll add it to our backlog for prioritisation.

@kentquirk
Copy link
Contributor

Thank you for the suggestion. Because this project is so specific to the use case, we think that OTel is not really a good match for its needs. There are specific things it does using Honeycomb's native events (the watch command for CircleCI, for example) that would be difficult to replicate in a generic OTel solution. Just letting you know that it's not likely to rise out of the icebox soon.

@dmathieu
Copy link

I think rather than using the otel library directly, what would be nice here is if buildevents would support OTLP.
Then it would still be able to start/finish traces its own way. But could send them to any OTLP-valid provider, or OpenTelemetry Collector.

@vreynolds vreynolds added the status: oncall Flagged for awareness from Honeycomb Telemetry Oncall label May 23, 2022
@MikeGoldsmith MikeGoldsmith removed the status: oncall Flagged for awareness from Honeycomb Telemetry Oncall label Jun 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants