-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add default Collector configuration path to Configuration docs #3940
Conversation
…m:open-telemetry/opentelemetry.io into theletterf-3929-add-collector-config-path
PTAL. I've made the override examples more realistic and edited the location paragraph to make it a bit more... flexible. |
```shell | ||
# The following example sets the verbosity | ||
# level of the debug exporter to 'detailed' | ||
otelcol --set "exporters::debug::verbosity=detailed" | ||
# The following example overrides gRPC | ||
# settings for the OTLP receiver | ||
otelcol --set "receivers::otlp::protocols::grpc={endpoint:localhost:4317, compression: gzip}" | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bogdandrutu do you think having these examples make sense?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If they're syntactically correct, could we merge and review later?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 let's do this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sweet! Could you approve? Thanks!
Fixes #3929. Preview: https://deploy-preview-3940--opentelemetry.netlify.app/docs/collector/configuration/
I've added two new sections:
--set
option.Most of the information was taken from https://github.com/open-telemetry/opentelemetry-collector/blob/fa1b24184cb20ed0584457ea68c49c908da5f988/service/README.md?plain=1#L5, assuming that it applies to the
otelcol
binary as well, though confirmation is required.Some questions:
otelcorecol
README file apply?Note: It'd be really, really great to revive open-telemetry/opentelemetry-collector#5654