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

Collector "Configuration" page issues #1697

Closed
3 tasks
svrnm opened this issue Sep 1, 2022 · 2 comments · Fixed by #3443
Closed
3 tasks

Collector "Configuration" page issues #1697

svrnm opened this issue Sep 1, 2022 · 2 comments · Fixed by #3443
Labels
bug Something isn't working docs help wanted Extra attention is needed sig:collector

Comments

@svrnm
Copy link
Member

svrnm commented Sep 1, 2022

  • The config under the Configuration page, Exporters section, is a bit odd. For instance, the prometheus exporter specifies prometheus:8889 as the endpoint. As it refers to the same host, I would use either localhost or 0.0.0.0. Similarly, external URLs are following different patterns: "jaeger-all-in-one", "some.url", "example.com", "localhost". I would suggest having them all follow a pattern like "jaeger.example.com", "prometheus.example.com", "otlp.example.com", and "zipkin.example.com"
  • The configuration page is missing the telemetry attribute for the service
  • A link is desirable for "net/http" mentioned in "Proxy support"

Source: #1368 (comment)

@svrnm svrnm added bug Something isn't working help wanted Extra attention is needed docs sig:collector labels Sep 1, 2022
@theletterf
Copy link
Member

theletterf commented Oct 2, 2023

I'd like to help with this.

@theletterf
Copy link
Member

@svrnm What do you mean by

The configuration page is missing the telemetry attribute for the service

You mean OTEL_SERVICE_NAME?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working docs help wanted Extra attention is needed sig:collector
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants