opentelemetry-metrics-sdk 0.10.0 → 0.10.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f1cb67fd12dc3335ced46799a0eec9891023f9e76da30ea07ad55b1211f66999
-  data.tar.gz: 940e0e1554ab9762fcca4ebe624bb00f2a8a72122d7ef1bd3e1c0c37691a979c
+  metadata.gz: 895b79f7aaf3ac2fb3a31145d87526f75287ed7187518991a7e83353f0dc95a9
+  data.tar.gz: 61464ed3f41e4cd72e9eff00a8db7d67b7a80abfb7bc75333f43a3baa089e0ab
 SHA512:
-  metadata.gz: 2421112176546e9d94203f61f385c78fa524c8b86d9d7ddd509ce8c5873529256b2d3c5201bb3571a92f8ab2db92ae3c47fade6c71a68e91ee7804c00e9c724c
-  data.tar.gz: 3161006402ff67e3d14418a0c2cfad65047b7a5d157b9b4d85fc1cb8ee3d468cf324ad7cb71d875c376cddafbc8541446de722d266aa41a2579a73c37a058956
+  metadata.gz: 2b5e601b78c3bb2735c8eb2105ed5e081e015a95336bcc4b2b578eedb301837ce8109a0dc5336b64d557e966882c7c3e688bdfa0b49ed85c8a9a4b89886a598e
+  data.tar.gz: a72fe896576ab3757fdce00d29c8fb05d4ec64f8d60b4c260f40a5295a7e9184feef78351d1915dcc98d371dfcad693aa5895988c57c0a28628f0f03388d6cdb

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Release History: opentelemetry-metrics-sdk
 
+### v0.10.1 / 2025-10-21
+
+* FIXED: Update callback timeout mechanism to use Thread
+
 ### v0.10.0 / 2025-10-14
 
 * ADDED: Use common method for returning timestamp in nanoseconds

data/lib/opentelemetry/sdk/metrics/state/asynchronous_metric_stream.rb

@@ -13,6 +13,8 @@ module OpenTelemetry
         # The AsynchronousMetricStream class provides SDK internal functionality that is not a part of the
         # public API. It extends MetricStream to support asynchronous instruments.
         class AsynchronousMetricStream < MetricStream
+          DEFAULT_TIMEOUT = 30
+
           def initialize(
             name,
             description,
@@ -48,28 +50,49 @@ module OpenTelemetry
           def invoke_callback(timeout, attributes)
             if @registered_views.empty?
               @mutex.synchronize do
-                Timeout.timeout(timeout || 30) do
-                  @callback.each do |cb|
-                    value = cb.call
-                    @default_aggregation.update(value, attributes, @data_points)
-                  end
+                @callback.each do |cb|
+                  value = safe_guard_callback(cb, timeout: timeout)
+                  @default_aggregation.update(value, attributes, @data_points) if value.is_a?(Numeric)
                 end
               end
             else
               @registered_views.each do |view|
                 @mutex.synchronize do
-                  Timeout.timeout(timeout || 30) do
-                    @callback.each do |cb|
-                      value = cb.call
-                      merged_attributes = attributes || {}
-                      merged_attributes.merge!(view.attribute_keys)
-                      view.aggregation.update(value, merged_attributes, @data_points) if view.valid_aggregation?
-                    end
+                  @callback.each do |cb|
+                    value = safe_guard_callback(cb, timeout: timeout)
+                    next unless value.is_a?(Numeric) # ignore if value is not valid number
+
+                    merged_attributes = attributes || {}
+                    merged_attributes.merge!(view.attribute_keys)
+                    view.aggregation.update(value, merged_attributes, @data_points) if view.valid_aggregation?
                   end
                 end
               end
             end
           end
+
+          private
+
+          def safe_guard_callback(callback, timeout: DEFAULT_TIMEOUT)
+            result = nil
+            thread = Thread.new do
+              result = callback.call
+            rescue StandardError => e
+              OpenTelemetry.logger.error("Error invoking callback: #{e.message}")
+              result = :error
+            end
+
+            unless thread.join(timeout)
+              thread.kill
+              OpenTelemetry.logger.error("Timeout while invoking callback after #{timeout} seconds")
+              return nil
+            end
+
+            result == :error ? nil : result
+          rescue StandardError => e
+            OpenTelemetry.logger.error("Unexpected error in callback execution: #{e.message}")
+            nil
+          end
         end
       end
     end

data/lib/opentelemetry/sdk/metrics/state/metric_stream.rb

@@ -55,6 +55,7 @@ module OpenTelemetry
             end
           end
 
+          # view will modify the data_point that is not suitable when there are multiple views
           def update(value, attributes)
             if @registered_views.empty?
               @mutex.synchronize { @default_aggregation.update(value, attributes, @data_points) }

data/lib/opentelemetry/sdk/metrics/version.rb

@@ -8,7 +8,7 @@ module OpenTelemetry
   module SDK
     module Metrics
       # Current OpenTelemetry metrics sdk version
-      VERSION = '0.10.0'
+      VERSION = '0.10.1'
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: opentelemetry-metrics-sdk
 version: !ruby/object:Gem::Version
-  version: 0.10.0
+  version: 0.10.1
 platform: ruby
 authors:
 - OpenTelemetry Authors
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-10-14 00:00:00.000000000 Z
+date: 2025-10-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: opentelemetry-api
@@ -251,10 +251,10 @@ homepage: https://github.com/open-telemetry/opentelemetry-ruby
 licenses:
 - Apache-2.0
 metadata:
-  changelog_uri: https://open-telemetry.github.io/opentelemetry-ruby/opentelemetry-metrics-sdk/v0.10.0/file.CHANGELOG.html
+  changelog_uri: https://open-telemetry.github.io/opentelemetry-ruby/opentelemetry-metrics-sdk/v0.10.1/file.CHANGELOG.html
   source_code_uri: https://github.com/open-telemetry/opentelemetry-ruby/tree/main/metrics_sdk
   bug_tracker_uri: https://github.com/open-telemetry/opentelemetry-ruby/issues
-  documentation_uri: https://open-telemetry.github.io/opentelemetry-ruby/opentelemetry-metrics-sdk/v0.10.0
+  documentation_uri: https://open-telemetry.github.io/opentelemetry-ruby/opentelemetry-metrics-sdk/v0.10.1
 post_install_message:
 rdoc_options: []
 require_paths: