Skip to content

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

Merged
heathseals merged 1 commit into
masterfrom
bump_otel_gems
Aug 24, 2021
Merged

(POOLER-198) Fix otel warning: Bump otel gems to 0.17.0#457
heathseals merged 1 commit into
masterfrom
bump_otel_gems

Conversation

@yachub

@yachub yachub commented Aug 23, 2021

Copy link
Copy Markdown

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-jha isaac-jha 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