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

[receivercreator] fix typo in examples #30227

Merged
merged 2 commits into from
Dec 29, 2023
Merged

[receivercreator] fix typo in examples #30227

merged 2 commits into from
Dec 29, 2023

Conversation

fatsheep9146
Copy link
Contributor

Description:
fix typo in examples, since the observe_nodes, observe_services is default to be false, so the receiver creator who wants to add endpoint of k8s.node and k8s.service will not work if the two config is false

@fatsheep9146 fatsheep9146 added documentation Improvements or additions to documentation Skip Changelog PRs that do not require a CHANGELOG.md entry receiver/receivercreator labels Dec 28, 2023
@fatsheep9146 fatsheep9146 requested review from a team and codeboten December 28, 2023 05:57
@codeboten codeboten merged commit 50a618e into main Dec 29, 2023
85 checks passed
@codeboten codeboten deleted the fatsheep9146-patch-1 branch December 29, 2023 01:22
@github-actions github-actions bot added this to the next release milestone Dec 29, 2023
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Jan 10, 2024
fix typo in examples, since the `observe_nodes`, `observe_services` is
[default](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/extension/observer/k8sobserver/README.md)
to be `false`, so the receiver creator who wants to add endpoint of
k8s.node and k8s.service will not work if the two config is `false`

---------

Co-authored-by: Alex Boten <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation receiver/receivercreator Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants