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

[Bug]: spanmetrics via connector not working with otel-collector v0.80.0 #4547

Closed
Multiply opened this issue Jun 22, 2023 · 3 comments · Fixed by #4555
Closed

[Bug]: spanmetrics via connector not working with otel-collector v0.80.0 #4547

Multiply opened this issue Jun 22, 2023 · 3 comments · Fixed by #4555
Labels

Comments

@Multiply
Copy link

What happened?

The duration metric name is no longer normalized with the unit type by default.
The feature gate has been reverted back to off by default, see more details here: https://github.com/open-telemetry/opentelemetry-collector-contrib/releases/tag/v0.80.0

Steps to reproduce

When trying to run the docker-compose monitor example, no data is visible in Jaeger's Monitor tab.

Expected behavior

I expect Jaeger to support both normalized and non-normalized metrics names, or at least document the need to have the feature gate enabled.

Relevant log output

No response

Screenshot

No response

Additional context

No response

Jaeger backend version

v1.46.0

SDK

OpenTelemetry Collector v0.80.0

Pipeline

The default provided in the docker-compose monitor example.

Stogage backend

No response

Operating system

No response

Deployment model

No response

Deployment configs

No response

@Multiply Multiply added the bug label Jun 22, 2023
@Multiply Multiply changed the title [Bug]: spanmetrics via connector not working with otel-contrib v0.80.0 [Bug]: spanmetrics via connector not working with otel-collector v0.80.0 Jun 22, 2023
@yurishkuro
Copy link
Member

@albertteoh

@albertteoh
Copy link
Contributor

Thanks for letting us know, @Multiply.

My mistake for not recognising the behaviour was caused by a feature gate in the first place!

Will take a look.

@Multiply
Copy link
Author

I'll be happy to test both locally and in production when something lands!

albertteoh added a commit that referenced this issue Jun 29, 2023
## Which problem is this PR solving?
- Resolves #4547

## Short description of the changes
- Adds the following boolean parameters to declaratively determine
whether the metric name should be modified to match normalization rules
as defined
[here](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/pkg/translator/prometheus/README.md):
  - `prometheus.query.normalize-calls`
  - `prometheus.query.normalize-duration`
- Separate flags intentional to allow support for backwards
compatibility with older OpenTelemetry Collector versions. A single flag
is insufficient.
- Motivated by a breaking change in:
https://github.com/open-telemetry/opentelemetry-collector-contrib/releases/tag/v0.80.0.

---------

Signed-off-by: albertteoh <see.kwang.teoh@gmail.com>
afzal442 pushed a commit to Cloud-Hacks/jaeger that referenced this issue Jul 10, 2023
## Which problem is this PR solving?
- Resolves jaegertracing#4547

## Short description of the changes
- Adds the following boolean parameters to declaratively determine
whether the metric name should be modified to match normalization rules
as defined
[here](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/pkg/translator/prometheus/README.md):
  - `prometheus.query.normalize-calls`
  - `prometheus.query.normalize-duration`
- Separate flags intentional to allow support for backwards
compatibility with older OpenTelemetry Collector versions. A single flag
is insufficient.
- Motivated by a breaking change in:
https://github.com/open-telemetry/opentelemetry-collector-contrib/releases/tag/v0.80.0.

---------

Signed-off-by: albertteoh <see.kwang.teoh@gmail.com>
Signed-off-by: Afzal Ansari <afzal442@gmail.com>
Monstergogo added a commit to Monstergogo/beauty-share that referenced this issue Nov 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants