Comment TraceLevelInfoFilter as it breaks OTel compatibility

[vikmik]

TraceLevelInfoFilter isn't used - its only use was in

gubernator/daemon.go

Line 116 in e054be3

// otelgrpc.WithInterceptorFilter(TraceLevelInfoFilter),

but is currently commented.

This PR unblocks consumers of gubernator who also use the latest OTel libraries (version 0.52), as the type otelgrpc.Filter changed in open-telemetry/opentelemetry-go-contrib#5196 ). See change here. The code I commented is now invalid with latest versions of OTel, which prevents consumers of OTel libraries to upgrade them if they also use gubernator. Building gubernator with the latest OTel libraries fails with:

# github.com/gubernator-io/gubernator/v2
../../../../pkg/mod/github.com/gubernator-io/gubernator/[email protected]/config.go:739:44: cannot convert func(info *otelgrpc.InterceptorInfo) bool {…} (value of type func(info *otelgrpc.InterceptorInfo) bool) to type otelgrpc.Filter

Allegedly this could now be fixed with the merge of the above PR (and the code in daemon.go could be uncommented), but it requires a larger change.

I am opening this PR to unblock users of OTel and gubernator as it is the most pressing issue.

---

Commenting feels a bit weird (I did it because of the commented code in daemon.go), let me know if you want to address this differently.

[vikmik]

Also cc @thrawn01 just in case :)

[thrawn01]

There have been so many golang mod compatibility errors with otel, over the past few months. I'm beginning to wonder if it's worth the hassle.

The maintainers of different otel libraries don't appear to be managing releases properly. 😞

[thrawn01]

For my Ref: open-telemetry/opentelemetry-go-contrib#4575 no deprecation notice, with no suggestions of an alternative. 👎

[thrawn01]

You can delete this code if you wish, but I'll accept the PR as is. If I don't hear back from you in the next 12 hrs, I'll merge.

Thank you for making this PR!

[vikmik]

Just seeing this. Thank you! Yes it seems that having an OTel dependency can make things difficult. I made this PR recently too. Thanks for merging this PR

[pbennett]

There have been so many golang mod compatibility errors with otel, over the past few months. I'm beginning to wonder if it's worth the hassle.

The maintainers of different otel libraries don't appear to be managing releases properly. 😞

I'm inclined to agree. I use some other packages that recently switched (somewhat forced) to otel as well and it seems like it might becoming a bit of a nightmare dependency.