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

(otelarrowreceiver): remove obsreporttest package #161

Merged

Conversation

moh-osman3
Copy link
Contributor

Remove a deprecated package that is removed completely from collector v0.97.0 (open-telemetry/opentelemetry-collector#9724). This seems to be blocking a collector upgrade.

Attempt to upgrade collector to v0.97.0 results in

go: found github.com/open-telemetry/otel-arrow/collector/receiver/filereceiver in github.com/open-telemetry/otel-arrow/collector/receiver/filereceiver v0.18.0
go: found github.com/open-telemetry/otel-arrow/collector/receiver/otelarrowreceiver in github.com/open-telemetry/otel-arrow/collector/receiver/otelarrowreceiver v0.18.0
go: finding module for package go.opentelemetry.io/collector/obsreport/obsreporttest
go: github.com/open-telemetry/otel-arrow/collector/cmd/otelarrowcol imports
        github.com/open-telemetry/otel-arrow/collector/receiver/otelarrowreceiver tested by
        github.com/open-telemetry/otel-arrow/collector/receiver/otelarrowreceiver.test imports
        go.opentelemetry.io/collector/obsreport/obsreporttest: module go.opentelemetry.io/collector@latest found (v0.97.0), but does not contain package go.opentelemetry.io/collector/obsreport/obsreporttest

@moh-osman3 moh-osman3 marked this pull request as ready for review March 26, 2024 20:06
@jmacd jmacd merged commit f0b76fa into open-telemetry:main Mar 26, 2024
2 checks passed
jmacd pushed a commit that referenced this pull request Mar 27, 2024
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.

None yet

3 participants