Hi Simon,

On Wed, 2024-01-24 at 11:41 +0100, Simon Josefsson wrote:
> Yeah, alas, golang-opentelemetry-contrib needs newer
> golang-opentelemetry-otel, otherwise you get build errors like this:
> 
> src/go.opentelemetry.io/contrib/config/trace.go:13:2: cannot find package 
> "go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc" in any of:
>       
> /usr/lib/go-1.21/src/go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc
>  (from $GOROOT)
>       
> /build/golang-opentelemetry-contrib-0.47.0/obj-x86_64-linux-gnu/src/go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc
>  (from $GOPATH)
> src/go.opentelemetry.io/contrib/config/trace.go:14:2: cannot find package 
> "go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp" in any of:
>       
> /usr/lib/go-1.21/src/go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp
>  (from $GOROOT)
>       
> /build/golang-opentelemetry-contrib-0.47.0/obj-x86_64-linux-gnu/src/go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp
>  (from $GOPATH)
> src/go.opentelemetry.io/contrib/config/trace.go:19:2: cannot find package 
> "go.opentelemetry.io/otel/trace/noop" in any of:
>       /usr/lib/go-1.21/src/go.opentelemetry.io/otel/trace/noop (from $GOROOT)
>       
> /build/golang-opentelemetry-contrib-0.47.0/obj-x86_64-linux-gnu/src/go.opentelemetry.io/otel/trace/noop
>  (from $GOPATH)
> src/go.opentelemetry.io/contrib/propagators/opencensus/binary.go:22:2: cannot 
> find package "go.opentelemetry.io/otel/bridge/opencensus" in any of:
>       /usr/lib/go-1.21/src/go.opentelemetry.io/otel/bridge/opencensus (from 
> $GOROOT)
>       
> /build/golang-opentelemetry-contrib-0.47.0/obj-x86_64-linux-gnu/src/go.opentelemetry.io/otel/bridge/opencensus
>  (from $GOPATH)
> 
> To have confidence in an update of otel, I first built reverse
> dependencies of otel in sid but there are failures:
> 
> https://salsa.debian.org/jas/golang-opentelemetry-otel/-/pipelines/629791
> 
> I think this really indicate FTBFS bugs in cadvisor, crowdsec, and
> gitlab-ci-multi-runner?  Rebuilding packages in sid should work.

I'll open bugs and try fix them when time permits.

> I will try to update otel to 1.23.0-rc.1 and see if I get it to build.
> Build failures on the three previous packages can be probably be
> attributed to FTBFS bugs in those three packages, rathern than
> incompatibility with otel 1.23.0-rc1.  But you don't know for certain...

Isn't 1.23.0-rc1 a pre-release? Sounds like d/watch needs to be fixed.
As soon as you get golang-opentelemetry-otel 1.23.0 building, could you please
upload it to experimental so I can build fixed packages against the new version?

BTW, thanks for the Salsa pipelines, they're very helpful in finding the
problems.

--
Kind regards,
Maytham (@Maytha8)

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to