opentelemetry-metrics-sdk 0.9.1 → 0.10.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 640af5c30dd7768db2a34210e70ec4119396363462fa7494f35148986c1cc09a
-  data.tar.gz: 73d7f52d0553d61e695c933e1fe38d39e9eff7b3c08002e4f9286470d575b65c
+  metadata.gz: 895b79f7aaf3ac2fb3a31145d87526f75287ed7187518991a7e83353f0dc95a9
+  data.tar.gz: 61464ed3f41e4cd72e9eff00a8db7d67b7a80abfb7bc75333f43a3baa089e0ab
 SHA512:
-  metadata.gz: 5b73aaf5a8193508dc2194206c29c969c2e21793b6986b592cb605cdd98992df8bd4cea571715d75ce6fe4fff980dbccff7cca913da486956115fbab3e3e691d
-  data.tar.gz: '08155ce74455222fd9d49a922578fa4171a9fe98be081751dcfc5c27d3bb58e9925124f9886686ae09ef68dcc7b9cd8165a805d0fcb02a87723748fe0f37ad27'
+  metadata.gz: 2b5e601b78c3bb2735c8eb2105ed5e081e015a95336bcc4b2b578eedb301837ce8109a0dc5336b64d557e966882c7c3e688bdfa0b49ed85c8a9a4b89886a598e
+  data.tar.gz: a72fe896576ab3757fdce00d29c8fb05d4ec64f8d60b4c260f40a5295a7e9184feef78351d1915dcc98d371dfcad693aa5895988c57c0a28628f0f03388d6cdb

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 # 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
+
 ### v0.9.1 / 2025-09-16
 
 * FIXED: Use mapping scale outside of rescale logic

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,
@@ -30,7 +32,7 @@ module OpenTelemetry
 
             # Initialize asynchronous-specific attributes
             @callback = callback
-            @start_time = now_in_nano
+            @start_time = OpenTelemetry::Common::Utilities.time_in_nanoseconds
             @timeout = timeout
             @attributes = attributes
           end
@@ -48,31 +50,48 @@ 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
 
-          def now_in_nano
-            (Time.now.to_r * 1_000_000_000).to_i
+          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

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

@@ -15,14 +15,14 @@ module OpenTelemetry
         class MetricStore
           def initialize
             @mutex = Mutex.new
-            @epoch_start_time = now_in_nano
+            @epoch_start_time = OpenTelemetry::Common::Utilities.time_in_nanoseconds
             @epoch_end_time = nil
             @metric_streams = []
           end
 
           def collect
             @mutex.synchronize do
-              @epoch_end_time = now_in_nano
+              @epoch_end_time = OpenTelemetry::Common::Utilities.time_in_nanoseconds
               snapshot = @metric_streams.flat_map { |ms| ms.collect(@epoch_start_time, @epoch_end_time) }
               @epoch_start_time = @epoch_end_time
 
@@ -36,12 +36,6 @@ module OpenTelemetry
               nil
             end
           end
-
-          private
-
-          def now_in_nano
-            (Time.now.to_r * 1_000_000_000).to_i
-          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.9.1'
+      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.9.1
+  version: 0.10.1
 platform: ruby
 authors:
 - OpenTelemetry Authors
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-09-16 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.9.1/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.9.1
+  documentation_uri: https://open-telemetry.github.io/opentelemetry-ruby/opentelemetry-metrics-sdk/v0.10.1
 post_install_message:
 rdoc_options: []
 require_paths: