-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[emailservice] Update Ruby version and bump dependencies #863
[emailservice] Update Ruby version and bump dependencies #863
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Causes some issues to start the service on mac M1
email-service | <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require': cannot load such file -- google/protobuf_c (LoadError)
email-service | Did you mean? google/protobuf
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from /usr/local/bundle/gems/google-protobuf-3.22.3-aarch64-linux/lib/google/protobuf.rb:51:in `rescue in <top (required)>'
email-service | from /usr/local/bundle/gems/google-protobuf-3.22.3-aarch64-linux/lib/google/protobuf.rb:48:in `<top (required)>'
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from /usr/local/bundle/gems/googleapis-common-protos-types-1.6.0/lib/google/rpc/status_pb.rb:4:in `<top (required)>'
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from /usr/local/bundle/gems/opentelemetry-exporter-otlp-0.24.0/lib/opentelemetry/exporter/otlp/exporter.rb:13:in `<top (required)>'
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from /usr/local/bundle/gems/opentelemetry-exporter-otlp-0.24.0/lib/opentelemetry/exporter/otlp.rb:7:in `<top (required)>'
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from email_server.rb:9:in `<main>'
email-service | <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require': Error loading shared library ld-linux-aarch64.so.1: No such file or directory (needed by /usr/local/bundle/gems/google-protobuf-3.22.3-aarch64-linux/lib/google/3.2/protobuf_c.so) - /usr/local/bundle/gems/google-protobuf-3.22.3-aarch64-linux/lib/google/3.2/protobuf_c.so (LoadError)
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from /usr/local/bundle/gems/google-protobuf-3.22.3-aarch64-linux/lib/google/protobuf.rb:49:in `<top (required)>'
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from /usr/local/bundle/gems/googleapis-common-protos-types-1.6.0/lib/google/rpc/status_pb.rb:4:in `<top (required)>'
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from /usr/local/bundle/gems/opentelemetry-exporter-otlp-0.24.0/lib/opentelemetry/exporter/otlp/exporter.rb:13:in `<top (required)>'
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from /usr/local/bundle/gems/opentelemetry-exporter-otlp-0.24.0/lib/opentelemetry/exporter/otlp.rb:7:in `<top (required)>'
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
email-service | from email_server.rb:9:in `<main>'
email-service exited with code 1
@mviitane would you be able to try it out again? |
Didn't help, same problem still. |
@mviitane borrowed a M1 to troubleshoot 😅 I've removed the
I still need to check how I can add that to the GH action in order to build it during releasae, as just But now we can build and run the updated email service. |
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
@mviitane can you test this one again to see if the latest fix Juliano made fixes this for you? |
My last update should have fixed it. I've tested in M1. But I won't be able to take a look at it so soon. |
Changes
This PR updates Ruby version to
3.2.2
and bumps dependencies to the latest.Merge Requirements
For new features contributions please make sure you have completed the following
essential items:
CHANGELOG.md
updated to document new feature additionsMaintainers will not merge until the above have been completed. If you're unsure
which docs need to be changed ping the
@open-telemetry/demo-approvers.