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

Fix wmi error on contrib-tests #3858

Merged
merged 2 commits into from
Aug 18, 2021

Conversation

mx-psi
Copy link
Member

@mx-psi mx-psi commented Aug 18, 2021

Description:

Fix wmi error on contrib tests by requiring the latest version.
This error was probably introduced by #3802.
This uncovers a different, unrelated error:

go test -race -timeout 30s ./...
# github.com/influxdata/influxdb-observability/common
/home/circleci/go/pkg/mod/github.com/influxdata/influxdb-observability/[email protected]/metrics_sort.go:20:35: m.Gauge().DataPoints().At(l).LabelsMap undefined (type pdata.NumberDataPoint has no field or method LabelsMap)
/home/circleci/go/pkg/mod/github.com/influxdata/influxdb-observability/[email protected]/metrics_sort.go:24:33: m.Sum().DataPoints().At(l).LabelsMap undefined (type pdata.NumberDataPoint has no field or method LabelsMap)
/home/circleci/go/pkg/mod/github.com/influxdata/influxdb-observability/[email protected]/metrics_sort.go:29:39: m.Histogram().DataPoints().At(l).LabelsMap undefined (type pdata.HistogramDataPoint has no field or method LabelsMap)
/home/circleci/go/pkg/mod/github.com/influxdata/influxdb-observability/[email protected]/metrics_sort.go:33:37: m.Summary().DataPoints().At(l).LabelsMap undefined (type pdata.SummaryDataPoint has no field or method LabelsMap)
FAIL	github.com/open-telemetry/opentelemetry-collector-contrib/cmd/otelcontribcol [build failed]
ok  	github.com/open-telemetry/opentelemetry-collector-contrib/internal/version	0.021s [no tests to run]
FAIL

Link to tracking Issue: relates to discussion on open-telemetry/opentelemetry-collector-contrib#4433.

@mx-psi mx-psi marked this pull request as ready for review August 18, 2021 16:43
@mx-psi mx-psi requested review from a team and tigrannajaryan August 18, 2021 16:43
go.mod Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants