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

[docs] add guidance around what a collector/distro is #12435

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[docs] add guidance around what a collector/distro is #12435

wants to merge 1 commit into from

Conversation

codeboten
Copy link
Contributor

Fixes #8555

@codeboten codeboten requested a review from a team as a code owner February 19, 2025 16:52
@austinlparker austinlparker mentioned this pull request Feb 19, 2025
Closed
@codeboten codeboten mentioned this pull request Feb 19, 2025
Closed
1 task
@codeboten codeboten added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Feb 19, 2025
@codecov Codecov
Copy link

codecov bot commented Feb 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.18%. Comparing base (c2af75d) to head (5efc10e).

Additional details and impacted files 
@@           Coverage Diff           @@
##             main   #12435   +/-   ##
=======================================
  Coverage   92.18%   92.18%           
=======================================
  Files         465      465           
  Lines       25275    25275           
=======================================
  Hits        23301    23301           
  Misses       1575     1575           
  Partials      399      399

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

jpkrohling
jpkrohling approved these changes Feb 19, 2025
View reviewed changes
Copy link
Member

@jpkrohling jpkrohling left a comment

Choose a reason for hiding this comment

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

Bringing my approval from the original PR.

songy23
songy23 approved these changes Feb 19, 2025
View reviewed changes
Copy link
Member

@songy23 songy23 left a comment

Choose a reason for hiding this comment

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

LGTM

jmacd
jmacd approved these changes Feb 21, 2025
View reviewed changes
Copy link
Contributor

@jmacd jmacd left a comment

Choose a reason for hiding this comment

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

🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jpkrohling jpkrohling jpkrohling approved these changes

@tedsuo tedsuo tedsuo approved these changes

@jmacd jmacd jmacd approved these changes

@austinlparker austinlparker austinlparker approved these changes

@mx-psi mx-psi mx-psi approved these changes

@jaronoff97 jaronoff97 jaronoff97 approved these changes

@songy23 songy23 songy23 approved these changes

@TylerHelmuth TylerHelmuth TylerHelmuth approved these changes

@bogdandrutu bogdandrutu Awaiting requested review from bogdandrutu bogdandrutu is a code owner automatically assigned from open-telemetry/collector-approvers

Assignees
No one assigned
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry Skip Contrib Tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

What is an OpenTelemetry Collector, what is a distribution?
9 participants
@codeboten @jpkrohling @tedsuo @jmacd @austinlparker @mx-psi @jaronoff97 @songy23 @TylerHelmuth