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

(POOLER-198) Fix otel warning: Bump otel gems to 0.17.0 #457

Merged
merged 1 commit into from
Aug 24, 2021

Conversation

yachub
Copy link

@yachub yachub commented Aug 23, 2021

Resolves the following errors related to the opentelemetry-sdk gem:

warning: thread "Ruby-0-Thread-1: /usr/local/bundle/gems/opentelemetry-sdk-0.15.0/lib/opentelemetry/sdk/trace/export/batch_span_processor.rb:179" terminated with exception (report_on_exception is true):
NoMethodError: undefined method `export' for #<Hash:0x61159131>
   export_batch at /usr/local/bundle/gems/opentelemetry-sdk-0.15.0/lib/opentelemetry/sdk/trace/export/batch_span_processor.rb:186
    synchronize at org/jruby/ext/thread/Mutex.java:164
   export_batch at /usr/local/bundle/gems/opentelemetry-sdk-0.15.0/lib/opentelemetry/sdk/trace/export/batch_span_processor.rb:186
           work at /usr/local/bundle/gems/opentelemetry-sdk-0.15.0/lib/opentelemetry/sdk/trace/export/batch_span_processor.rb:169
           loop at org/jruby/RubyKernel.java:1442
           work at /usr/local/bundle/gems/opentelemetry-sdk-0.15.0/lib/opentelemetry/sdk/trace/export/batch_span_processor.rb:158
  reset_on_fork at /usr/local/bundle/gems/opentelemetry-sdk-0.15.0/lib/opentelemetry/sdk/trace/export/batch_span_processor.rb:179
2021-08-21 13:01:45 - NoMethodError - undefined method `each' for "DEUEUQ":String:
	/usr/local/bundle/gems/vmpooler-1.1.0/lib/vmpooler/api/v1.rb:145:in `block in fetch_single_vm'

And addresses breaking changes due to open-telemetry/opentelemetry-ruby#600 (should have released in 0.14.0?)

Full changelog from 0.15.0 to 0.17.0: open-telemetry/opentelemetry-ruby@opentelemetry-sdk/v0.15.0...opentelemetry-sdk/v0.17.0

@yachub yachub changed the title Bump otel gems to 0.17.0 Fix otel warning: Bump otel gems to 0.17.0 Aug 23, 2021
@yachub yachub marked this pull request as ready for review August 24, 2021 12:57
@yachub yachub requested a review from a team as a code owner August 24, 2021 12:57
@heathseals heathseals merged commit 9b81aa1 into master Aug 24, 2021
@yachub yachub changed the title Fix otel warning: Bump otel gems to 0.17.0 (DIO-2456) Fix otel warning: Bump otel gems to 0.17.0 Aug 24, 2021
@yachub yachub changed the title (DIO-2456) Fix otel warning: Bump otel gems to 0.17.0 (POOLER-198) Fix otel warning: Bump otel gems to 0.17.0 Aug 24, 2021
@isaac-hammes isaac-hammes deleted the bump_otel_gems branch August 18, 2023 16:11
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

Successfully merging this pull request may close these issues.

2 participants