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

Found the error go.opentelemetry.io/otel/sdk/export for opentelemetry with jaeger exporter #470

Closed
iamgoangle opened this issue Feb 9, 2020 · 4 comments

Comments

@iamgoangle
Copy link

iamgoangle commented Feb 9, 2020

I am trying to run the example https://github.com/open-telemetry/opentelemetry-go/blob/master/example/jaeger/main.go

trying to get all dependencies
image

I got an error like this
image

Would you please help? This is my repositoty https://github.com/iamgoangle/opentelemetry-jaeger-exporter

@wingyplus
Copy link
Contributor

wingyplus commented Feb 10, 2020

Seems like it fetch jaeger exporter as a v1.0.0.

$ go get -v go.opentelemetry.io/otel/exporter/trace/jaeger
go: finding versions for go.opentelemetry.io/otel/exporter/trace
go: finding versions for go.opentelemetry.io/otel/exporter
go: finding versions for go.opentelemetry.io
go: go.opentelemetry.io/otel/exporter/trace/jaeger upgrade => v1.0.0
go: finding versions for go.opentelemetry.io/otel/sdk/export
go: finding versions for go.opentelemetry.io/otel/sdk
../../../../pkg/mod/go.opentelemetry.io/otel/exporter/trace/jaeger@v1.0.0/jaeger.go:27:2: module go.opentelemetry.io/otel@latest found (v0.2.1), but does not contain package go.opentelemetry.io/otel/sdk/export

Not sure why it's upgrade to v1.0.0. So I just try the workaround by mark a version to v0.2.1 during go get jaeger exporter

$ go get -v go.opentelemetry.io/otel/exporter/trace/jaeger@v0.2.1

@wingyplus
Copy link
Contributor

It's seems to be a bad release on golang module proxy. Try it with GOPROXY=direct is work fine.

$ env GOPROXY=direct sh try.sh
go: creating new go.mod: module github.com/wingyplus/repro
get "go.opentelemetry.io/otel": found meta tag get.metaImport{Prefix:"go.opentelemetry.io/otel", VCS:"git", RepoRoot:"https://github.com/open-telemetry/opentelemetry-go"} at //go.opentelemetry.io/otel?go-get=1
go: finding versions for go.opentelemetry.io/otel
go: go.opentelemetry.io/otel upgrade => v0.2.1
can't load package: package go.opentelemetry.io/otel: build constraints exclude all Go files in /Users/wingyplus/pkg/mod/go.opentelemetry.io/otel@v0.2.1
get "go.opentelemetry.io/otel": found meta tag get.metaImport{Prefix:"go.opentelemetry.io/otel", VCS:"git", RepoRoot:"https://github.com/open-telemetry/opentelemetry-go"} at //go.opentelemetry.io/otel?go-get=1
get "go.opentelemetry.io/otel/exporter/trace": found meta tag get.metaImport{Prefix:"go.opentelemetry.io/otel", VCS:"git", RepoRoot:"https://github.com/open-telemetry/opentelemetry-go"} at //go.opentelemetry.io/otel/exporter/trace?go-get=1
get "go.opentelemetry.io/otel/exporter/trace": verifying non-authoritative meta tag
get "go.opentelemetry.io/otel/exporter/trace/jaeger": found meta tag get.metaImport{Prefix:"go.opentelemetry.io/otel", VCS:"git", RepoRoot:"https://github.com/open-telemetry/opentelemetry-go"} at //go.opentelemetry.io/otel/exporter/trace/jaeger?go-get=1
get "go.opentelemetry.io/otel/exporter/trace/jaeger": verifying non-authoritative meta tag
get "go.opentelemetry.io/otel/exporter": found meta tag get.metaImport{Prefix:"go.opentelemetry.io/otel", VCS:"git", RepoRoot:"https://github.com/open-telemetry/opentelemetry-go"} at //go.opentelemetry.io/otel/exporter?go-get=1
get "go.opentelemetry.io/otel/exporter": verifying non-authoritative meta tag
go: finding versions for go.opentelemetry.io/otel
go: finding versions for go.opentelemetry.io/otel/exporter/trace/jaeger
go: finding versions for go.opentelemetry.io/otel/exporter
go: finding versions for go.opentelemetry.io/otel/exporter/trace
go: finding go.opentelemetry.io/otel/exporter latest
go: finding go.opentelemetry.io/otel/exporter/trace latest
go: go.opentelemetry.io/otel/exporter/trace/jaeger upgrade => v0.2.1
$ cat try.sh
go mod init
go get -v go.opentelemetry.io/otel
go get -v go.opentelemetry.io/otel/exporter/trace/jaeger

@rghetia
Copy link
Contributor

rghetia commented Feb 13, 2020

@krnowak suggested to rename exporter to exporters. This will affect all exporters but it is okay since it is not GA.

@rghetia rghetia added this to the Alpha v0.3 milestone Feb 13, 2020
@rghetia
Copy link
Contributor

rghetia commented Feb 13, 2020

will be addressed by #331

@rghetia rghetia closed this as completed Feb 13, 2020
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

No branches or pull requests

3 participants