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

Bump github.com/DataDog/opentelemetry-mapping-go/pkg/otlp/metrics from 0.2.1 to 0.2.3 causes issue #22917

Closed
codeboten opened this issue May 29, 2023 · 5 comments · Fixed by #22952
Closed

Bump github.com/DataDog/opentelemetry-mapping-go/pkg/otlp/metrics from 0.2.1 to 0.2.3 causes issue #22917

codeboten opened this issue May 29, 2023 · 5 comments · Fixed by #22952
Assignees
@songy23
Labels
exporter/datadog Datadog components priority:p1 High

Comments

@codeboten
Copy link
Contributor

Component(s)

exporter/datadog

Describe the issue you're reporting

See https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/5114664886/jobs/9195143629#step:5:189 for more info:

Error: ../../exporter/datadogexporter/metrics_exporter.go:189:9: assignment mismatch: 1 variable but exp.tr.MapMetrics returns 2 values
@codeboten codeboten added the exporter/datadog Datadog components label May 29, 2023
@codeboten codeboten mentioned this issue May 29, 2023
Closed
@github-actions
Copy link
Contributor

Pinging code owners for exporter/datadog: @mx-psi @gbbr @dineshg13 @liustanley @songy23. See Adding Labels via Comments if you do not have permissions to add labels yourself.

@github-actions
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@mx-psi
Copy link
Member

mx-psi commented May 30, 2023

@liustanley can you take a look at this? It relates to DataDog/opentelemetry-mapping-go/pull/81

@mx-psi mx-psi added the priority:p1 High label May 30, 2023
@songy23
Copy link
Member

songy23 commented May 30, 2023

Stanley is out today, I'll take a look

@songy23 songy23 assigned songy23 and unassigned liustanley May 30, 2023
@codeboten
Copy link
Contributor Author

Thanks @mx-psi @songy23!

@songy23 songy23 mentioned this issue May 30, 2023
Merged
songy23 added a commit to songy23/opentelemetry-service-contrib that referenced this issue May 30, 2023
@songy23
[exporter/datadogexporter] Upgrade opentelemetry-mapping-go version a…
acfe554 
…nd fix breaking changes

Fixes open-telemetry#22917
codeboten pushed a commit that referenced this issue May 30, 2023
@songy23
[exporter/datadogexporter] Upgrade opentelemetry-mapping-go version a…
84429d1 
…nd fix breaking changes (#22952)

Upgrade opentelemetry-mapping-go version and fix breaking changes

Link to tracking Issue:
Fixes #22917
Caleb-Hurshman pushed a commit to observIQ/opentelemetry-collector-contrib that referenced this issue Jul 6, 2023
@songy23 @Caleb-Hurshman
[exporter/datadogexporter] Upgrade opentelemetry-mapping-go version a…
e269c14 
…nd fix breaking changes (open-telemetry#22952)

Upgrade opentelemetry-mapping-go version and fix breaking changes

Link to tracking Issue:
Fixes open-telemetry#22917
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@songy23 songy23

Labels
exporter/datadog Datadog components priority:p1 High
Projects
None yet
Milestone
No milestone
4 participants
@codeboten @mx-psi @songy23 @liustanley