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

feat: add honeycomb marker exporter structure #27001

Merged
merged 76 commits into from
Oct 12, 2023

Conversation

fchikwekwe
Copy link
Contributor

@fchikwekwe fchikwekwe commented Sep 19, 2023

Description: This component will export markers to be consumed by the Honeycomb Markers API to highlight user events based initially on preset configurations.

Link to tracking Issue: #26653

Testing: Unit testing for factory and config

Documentation: README describing component usage

@TylerHelmuth TylerHelmuth self-assigned this Sep 19, 2023
exporter/honeycombmarkerexporter/internal/logs_exporter.go Outdated Show resolved Hide resolved
exporter/honeycombmarkerexporter/factory.go Outdated Show resolved Hide resolved
exporter/honeycombmarkerexporter/config.go Outdated Show resolved Hide resolved
exporter/honeycombmarkerexporter/internal/logs_exporter.go Outdated Show resolved Hide resolved
exporter/honeycombmarkerexporter/internal/logs_exporter.go Outdated Show resolved Hide resolved
exporter/honeycombmarkerexporter/factory.go Outdated Show resolved Hide resolved
@fchikwekwe fchikwekwe marked this pull request as ready for review September 25, 2023 16:33
@fchikwekwe fchikwekwe requested review from a team and mx-psi September 25, 2023 16:33
Copy link
Contributor

@MovieStoreGuy MovieStoreGuy left a comment

Choose a reason for hiding this comment

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

Hey, just a few things that I can point out that would need addressing.

Otherwise, I will leave it to @TylerHelmuth to give the final approval.

exporter/honeycombmarkerexporter/config.go Outdated Show resolved Hide resolved
exporter/honeycombmarkerexporter/README.md Outdated Show resolved Hide resolved
exporter/honeycombmarkerexporter/factory.go Outdated Show resolved Hide resolved
exporter/honeycombmarkerexporter/metadata.yaml Outdated Show resolved Hide resolved
exporter/honeycombmarkerexporter/config.go Outdated Show resolved Hide resolved
exporter/honeycombmarkerexporter/factory.go Outdated Show resolved Hide resolved
exporter/honeycombmarkerexporter/factory.go Outdated Show resolved Hide resolved
exporter/honeycombmarkerexporter/config.go Show resolved Hide resolved
fchikwekwe and others added 6 commits October 3, 2023 11:49
Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com>
Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com>
Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com>
mx-psi
mx-psi previously requested changes Oct 3, 2023
exporter/common/config.go Outdated Show resolved Hide resolved
@fchikwekwe fchikwekwe requested a review from mx-psi October 5, 2023 15:05
@fchikwekwe fchikwekwe force-pushed the feat/marker-exporter branch from 5b41859 to 7d6d0c5 Compare October 11, 2023 15:04
@fchikwekwe fchikwekwe requested a review from mx-psi October 12, 2023 13:56
@TylerHelmuth TylerHelmuth merged commit 628a6cb into open-telemetry:main Oct 12, 2023
82 checks passed
@github-actions github-actions bot added this to the next release milestone Oct 12, 2023
JaredTan95 pushed a commit to openinsight-proj/opentelemetry-collector-contrib that referenced this pull request Oct 18, 2023
**Description:** This component will export markers to be consumed by
the Honeycomb Markers API to highlight user events based initially on
preset configurations.

**Link to tracking Issue:** open-telemetry#26653 

**Testing:** Unit testing for factory and config

**Documentation:** README describing component usage

---------

Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com>
jmsnll pushed a commit to jmsnll/opentelemetry-collector-contrib that referenced this pull request Nov 12, 2023
**Description:** This component will export markers to be consumed by
the Honeycomb Markers API to highlight user events based initially on
preset configurations.

**Link to tracking Issue:** open-telemetry#26653 

**Testing:** Unit testing for factory and config

**Documentation:** README describing component usage

---------

Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
processor/filter Filter processor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants