kruby 1.36.0.3 → 1.36.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2368426a514cb2faa1a955a344c27535fd8a5e0ed658daa220ac3670b500a7dc
-  data.tar.gz: 5df9b8da264349a436a6b57568f165608c2a2630475ed27280f1c5b3c8fdfd5f
+  metadata.gz: a427ec5d2c6e70a8edf267a9095e38b988bfef3501dc420145de416fd8aea1aa
+  data.tar.gz: 6ddad098a7d16fde73bf1861b2aa4c1b4dd7559821b4d238bd58474a28bc1a99
 SHA512:
-  metadata.gz: 40a65006935258583abbe2b9111a43f6f5fa9022a7783a24c9166e0e03e9983b1243303e00d0903b893e99de105464c48b52f3040244c6cc8b02c0cab46f8c82
-  data.tar.gz: 7a8d5684829a67ece0845578f14c2caee7f92df7a4360dc6afeb265a35ddff1803d3741f811530a8f314f59b8fac2d632ff8760cd80db855f89d8aa2a7f89fc8
+  metadata.gz: 1ddbe0af5eaa1ae160af1e07d7c9b54209dd064fc20362a81e93b8f709ba01feb6f4835995652db3984d27b46501cf029a3b9dfbc87252175638392f40feb761
+  data.tar.gz: 5959a6ec6db11ae3e323f6759add33c9c9a57cde7d1f5727c37138f2603129b2814ec5e803fe015289e9d60be2c551e135ce58353c380cf27e310e1b0ed4ab59

data/lib/kubernetes/api_client.rb

@@ -15,10 +15,13 @@ require 'json'
 require 'logger'
 require 'tempfile'
 require 'time'
+require 'thread'
 require 'typhoeus'
 
 module Kubernetes
   class ApiClient
+    @default_mutex = Mutex.new
+
     # The Configuration object holding settings to be used in the API client.
     attr_accessor :config
 
@@ -39,7 +42,17 @@ module Kubernetes
     end
 
     def self.default
-      @@default ||= ApiClient.new
+      return @@default if defined?(@@default) && @@default
+
+      @default_mutex.synchronize do
+        @@default ||= ApiClient.new
+      end
+    end
+
+    def self.reset_default
+      @default_mutex.synchronize do
+        @@default = nil
+      end
     end
 
     # Call an API with given options.
@@ -47,34 +60,43 @@ module Kubernetes
     # @return [Array<(Object, Integer, Hash)>] an array of 3 elements:
     #   the data deserialized from response body (could be nil), response status code and response headers.
     def call_api(http_method, path, opts = {})
-      request = build_request(http_method, path, opts)
-      response = request.run
+      retries_performed = 0
 
-      if @config.debugging
-        @config.logger.debug "HTTP response body ~BEGIN~\n#{response.body}\n~END~\n"
-      end
+      loop do
+        request = build_request(http_method, path, opts)
+        response = request.run
 
-      unless response.success?
-        if response.timed_out?
-          fail ApiError.new('Connection timed out')
-        elsif response.code == 0
-          # Errors from libcurl will be made visible here
-          fail ApiError.new(:code => 0,
-                            :message => response.return_message)
-        else
-          fail ApiError.new(:code => response.code,
-                            :response_headers => response.headers,
-                            :response_body => response.body),
-               response.status_message
+        if @config.debugging
+          @config.logger.debug "HTTP response body ~BEGIN~\n#{response.body}\n~END~\n"
         end
-      end
 
-      if opts[:return_type]
-        data = deserialize(response, opts[:return_type])
-      else
-        data = nil
+        unless response.success?
+          if retry_request?(response, retries_performed)
+            interval_seconds = retry_interval_seconds(response, retries_performed)
+            log_retry_attempt(response, retries_performed + 1, interval_seconds)
+            sleep interval_seconds
+            retries_performed += 1
+            next
+          end
+
+          if response.timed_out?
+            fail ApiError.new(:code => 0,
+                              :message => 'Connection timed out')
+          elsif response.code == 0
+            # Errors from libcurl will be made visible here
+            fail ApiError.new(:code => 0,
+                              :message => response.return_message)
+          else
+            fail ApiError.new(:code => response.code,
+                              :response_headers => response.headers,
+                              :response_body => response.body),
+                 response.status_message
+          end
+        end
+
+        data = opts[:return_type] ? deserialize(response, opts[:return_type]) : nil
+        return data, response.code, response.headers
       end
-      return data, response.code, response.headers
     end
 
     # Builds the HTTP request
@@ -225,7 +247,12 @@ module Kubernetes
       # ensuring a default content type
       content_type = response.headers['Content-Type'] || 'application/json'
 
-      fail "Content-Type is not supported: #{content_type}" unless json_mime?(content_type)
+      unless json_mime?(content_type)
+        fail ApiError.new(:code => response.code,
+                          :response_headers => response.headers,
+                          :response_body => body,
+                          :message => "Content-Type is not supported: #{content_type}")
+      end
 
       begin
         data = JSON.parse("[#{body}]", :symbolize_names => true)[0]
@@ -399,5 +426,85 @@ module Kubernetes
         fail "unknown collection format: #{collection_format.inspect}"
       end
     end
+
+    def retry_request?(response, retries_performed)
+      return false if response.timed_out?
+      return false if response.code.to_i == 0
+      return false if max_retry_attempts <= 0
+      return false if retries_performed >= max_retry_attempts
+
+      retry_statuses.include?(response.code.to_i)
+    end
+
+    def retry_interval_seconds(response, retries_performed)
+      exponential_interval = retry_base_interval_seconds * (2**retries_performed)
+      server_interval = server_retry_after_seconds(response)
+
+      server_interval ? [exponential_interval, server_interval].max : exponential_interval
+    end
+
+    def log_retry_attempt(response, retry_number, interval_seconds)
+      @config.logger.info(
+        "Retrying API request after HTTP #{response.code} (retry #{retry_number}/#{max_retry_attempts}) in #{interval_seconds} seconds"
+      )
+      return unless @config.debugging
+
+      @config.logger.debug("Retry response body: #{response.body}")
+    end
+
+    def max_retry_attempts
+      value = fetch_retry_configuration(:max_retries, 0)
+      [value.to_i, 0].max
+    end
+
+    def retry_base_interval_seconds
+      [fetch_retry_configuration(:base_interval_seconds, 1.0).to_f, 0.0].max
+    end
+
+    def retry_statuses
+      Array(fetch_retry_configuration(:retry_statuses, [429, 500, 501, 502, 503])).map(&:to_i)
+    end
+
+    def server_retry_after_seconds(response)
+      header_value = response.headers['Retry-After'] if response.respond_to?(:headers) && response.headers.is_a?(Hash)
+      header_seconds = parse_retry_after_header(header_value)
+      return header_seconds if header_seconds
+
+      parse_retry_after_from_body(response.body)
+    end
+
+    def parse_retry_after_header(header_value)
+      return nil if header_value.nil?
+
+      integer_value = Integer(header_value, exception: false)
+      return integer_value.to_f if integer_value
+
+      retry_at = Time.httpdate(header_value)
+      [retry_at - Time.now, 0.0].max
+    rescue ArgumentError
+      nil
+    end
+
+    def parse_retry_after_from_body(body)
+      return nil if body.nil? || body.empty?
+
+      parsed = JSON.parse(body)
+      retry_after = parsed.dig('details', 'retryAfterSeconds') || parsed.dig(:details, :retryAfterSeconds)
+      return nil if retry_after.nil?
+
+      retry_after.to_f
+    rescue JSON::ParserError, TypeError
+      nil
+    end
+
+    def fetch_retry_configuration(key, default)
+      return default unless @config.retry_configuration.is_a?(Hash)
+
+      if @config.retry_configuration.key?(key)
+        @config.retry_configuration[key]
+      else
+        @config.retry_configuration.fetch(key.to_s, default)
+      end
+    end
   end
 end

data/lib/kubernetes/configuration.rb

@@ -11,9 +11,12 @@ OpenAPI Generator version: 5.1.0
 =end
 
 require 'logger'
+require 'thread'
 
 module Kubernetes
   class Configuration
+    @default_mutex = Mutex.new
+
     # Defines url scheme
     attr_accessor :scheme
 
@@ -88,6 +91,11 @@ module Kubernetes
     # Default to 0 (never times out).
     attr_accessor :timeout
 
+    # Optional retry configuration for retriable HTTP responses.
+    # Example:
+    #   { max_retries: 4, base_interval_seconds: 1.0, retry_statuses: [429, 500, 501, 502, 503] }
+    attr_accessor :retry_configuration
+
     # Set this to false to skip client side validation in the operation.
     # Default to true.
     # @return [true, false]
@@ -150,6 +158,7 @@ module Kubernetes
       @api_key = {}
       @api_key_prefix = {}
       @timeout = 0
+      @retry_configuration = nil
       @client_side_validation = true
       @verify_ssl = true
       @verify_ssl_host = true
@@ -166,7 +175,17 @@ module Kubernetes
 
     # The default Configuration object.
     def self.default
-      @@default ||= Configuration.new
+      return @@default if defined?(@@default) && @@default
+
+      @default_mutex.synchronize do
+        @@default ||= Configuration.new
+      end
+    end
+
+    def self.reset_default
+      @default_mutex.synchronize do
+        @@default = nil
+      end
     end
 
     # Backward-compatible alias used by examples and older integrations.

data/lib/kubernetes/utils.rb

@@ -15,6 +15,7 @@
 require 'kubernetes/config/incluster_config'
 require 'kubernetes/config/kube_config'
 require 'rubygems/version'
+require 'thread'
 
 # The Kubernetes module encapsulates the Kubernetes client for Ruby
 module Kubernetes
@@ -73,18 +74,22 @@ module Kubernetes
   end
 
   @temp_files = {}
+  @temp_files_mutex = Mutex.new
 
   def create_temp_file_with_base64content(content)
-    @temp_files[content] ||= Tempfile.open('kube') do |temp|
-      temp.write(Base64.strict_decode64(content))
-      temp
+    @temp_files_mutex.synchronize do
+      @temp_files[content] ||= Tempfile.open('kube') do |temp|
+        temp.write(Base64.strict_decode64(content))
+        temp
+      end
+      @temp_files.fetch(content).path
     end
-
-    @temp_files[content].path
   end
 
   def clear_temp_files
-    @temp_files = {}
+    @temp_files_mutex.synchronize do
+      @temp_files = {}
+    end
   end
 
   SUPPORTED_KUBERNETES_VERSION_RANGE = (

data/lib/kubernetes/version.rb

@@ -11,5 +11,5 @@ OpenAPI Generator version: 5.1.0
 =end
 
 module Kubernetes
-  VERSION = '1.36.0.3'
+  VERSION = '1.36.0.4'
 end

data/lib/kubernetes/watch.rb

@@ -20,6 +20,9 @@ module Kubernetes
   # The Watch class provides the ability to watch a specific resource for
   # updates.
   class Watch
+    MAX_RECONNECT_ATTEMPTS = 3
+    RECONNECT_BACKOFF_SECONDS = 1.0
+
     def initialize(client)
       @client = client
     end
@@ -34,24 +37,50 @@ module Kubernetes
     end
 
     def connect(path, resource_version = nil, &_block)
-      opts = { auth_names: ['BearerToken'] }
-      url = make_url(path, resource_version)
-      request = @client.build_request('GET', url, opts)
-      last = ''
-      request.on_body do |chunk|
-        last, pieces = split_lines(last, chunk)
-        pieces.each do |part|
-          begin
-            event = JSON.parse(part)
-          rescue JSON::ParserError => e
-            warn "Failed to parse watch event: #{e.message}. Raw event: #{part.inspect}"
+      current_resource_version = resource_version
+      reconnect_attempts = 0
+
+      loop do
+        reconnect_requested = false
+        opts = { auth_names: ['BearerToken'] }
+        url = make_url(path, current_resource_version)
+        request = @client.build_request('GET', url, opts)
+        last = ''
+
+        process_event = lambda do |part|
+          return if part.nil? || part.strip.empty?
+
+          event = parse_event(part)
+          return unless event
+
+          if watch_reset_event?(event)
+            current_resource_version = nil
+            reconnect_requested = true
             next
           end
 
+          current_resource_version = extract_resource_version(event) || current_resource_version
           yield event
         end
+
+        request.on_body do |chunk|
+          last, pieces = split_lines(last, chunk)
+          pieces.each { |part| process_event.call(part) }
+        end
+
+        request.on_complete do |_response|
+          process_event.call(last)
+          last = ''
+        end
+
+        response = request.run
+        reconnect_reason = reconnect_reason(response, reconnect_requested)
+        return response unless reconnect_reason
+        raise_terminal_watch_error(response, reconnect_reason) if reconnect_attempts >= MAX_RECONNECT_ATTEMPTS
+
+        reconnect_attempts += 1
+        sleep reconnect_backoff_seconds(reconnect_attempts - 1) if reconnect_reason == :transport
       end
-      request.run
     end
 
     def split_lines(last, chunk)
@@ -65,5 +94,51 @@ module Kubernetes
       last = data[(ix + 1)..data.length]
       [last, complete.split(/\n/)]
     end
+
+    private
+
+    def parse_event(part)
+      JSON.parse(part)
+    rescue JSON::ParserError => e
+      warn "Failed to parse watch event: #{e.message}. Raw event: #{part.inspect}"
+      nil
+    end
+
+    def extract_resource_version(event)
+      object = event['object']
+      return nil unless object.is_a?(Hash)
+
+      metadata = object['metadata']
+      return metadata['resourceVersion'] if metadata.is_a?(Hash)
+
+      object['resourceVersion']
+    end
+
+    def watch_reset_event?(event)
+      event['type'] == 'ERROR' && event.dig('object', 'code').to_i == 410
+    end
+
+    def reconnect_reason(response, reconnect_requested)
+      return :reset if reconnect_requested
+      return nil unless response
+
+      return :transport if response.timed_out? || response.code.to_i.zero?
+
+      nil
+    end
+
+    def reconnect_backoff_seconds(reconnect_attempt)
+      RECONNECT_BACKOFF_SECONDS * (2**reconnect_attempt)
+    end
+
+    def raise_terminal_watch_error(response, reconnect_reason)
+      if reconnect_reason == :reset
+        raise ApiError.new(code: 410, message: 'Watch resource version expired')
+      elsif response&.timed_out?
+        raise ApiError.new('Watch connection timed out')
+      elsif response && response.code.to_i.zero?
+        raise ApiError.new(code: 0, message: response.return_message)
+      end
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: kruby
 version: !ruby/object:Gem::Version
-  version: 1.36.0.3
+  version: 1.36.0.4
 platform: ruby
 authors:
 - doridoridoriand