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

tapioca dsl crashes with weird error when Gemfile contains OpenTelemetry gems. #1743

Closed
TwoAbove opened this issue Dec 22, 2023 · 3 comments
Closed

Comments

@TwoAbove
Copy link

Hey! I'm instrumenting a ruby rails app with OpenTelemetry, and I noticed that just adding the

gem 'opentelemetry-sdk'
gem 'opentelemetry-exporter-otlp'
gem 'opentelemetry-instrumentation-all'

to the Gemfile causes tapioca dsl to crash with this weird error:

> bundle exec tapioca dsl
...
Error: `Tapioca::Dsl::Compilers::Protobuf` failed to generate RBI for `Google::Protobuf::AbstractMessage`
bundler: failed to load command: tapioca (/Users/seva/.asdf/installs/ruby/3.2.2/bin/tapioca)
/Users/seva/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/tapioca-0.10.5/lib/tapioca/dsl/compilers/protobuf.rb:148:in `block in decorate': TypeError: Unexpected descriptor class: NilClass (Parallel::UndumpableException)
        from /Users/seva/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/tapioca-0.10.5/lib/tapioca/rbi_ext/model.rb:40:in `block in create_class'
        from <internal:kernel>:90:in `tap'
...

(full stack trace here)

The weird thing is that we don't use google-protobuf, and adding it to the Gemfile doesn't help either.

Removing the OpenTelemetry gems from the Gemfile allows tapioca dsl to succeed.

I'm at a loss here as to what to do. I tried to find ways to ignore the OpenTelemetry gems, but couldn't find anything that works.

Any suggestions to what I can do here?

Thanks!

@TwoAbove
Copy link
Author

I did find this #1411 similar issue

@TwoAbove
Copy link
Author

Update: I noticed that the tapoica version in that issue was higher than ours, so I'll report back after I update tapioca 😅

@TwoAbove
Copy link
Author

TwoAbove commented Jan 2, 2024

Looks like just updating helped!

@TwoAbove TwoAbove closed this as completed Jan 2, 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

No branches or pull requests

1 participant