Version 3.8.1

Author: mvallati

CHANGELOG.md

@@ -1,5 +1,9 @@
 ## next
 
+## 3.8.1
+
+- check if observedGeneration is present before comparing with generation for custom resources
+
 ## 3.8.0
 
 - Add Elasticsearch resource to the black list so it is not marked for deletion

lib/krane/kubernetes_resource/custom_resource.rb

@@ -39,7 +39,7 @@ def failure_message
     end
 
     def timeout_message
-      if rollout_conditions && current_generation != observed_generation
+      if observed_generation.present? && rollout_conditions && current_generation != observed_generation
         TIMEOUT_MESSAGE_DIFFERENT_GENERATIONS
       else
         super

lib/krane/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module Krane
-  VERSION = "3.8.0"
+  VERSION = "3.8.1"
 end

test/unit/krane/kubernetes_resource/custom_resource_definition_test.rb

@@ -159,11 +159,20 @@ def test_instance_timeout_annotation
   end
 
   def test_instance_timeout_messages_with_rollout_conditions
+    rollout_conditions = {
+      success_conditions: [
+        {
+          path: "$.status.conditions[?(@.type == 'Ready')].status",
+          value: "True",
+        },
+      ],
+    }.to_json
+
     crd = build_crd(crd_spec.merge(
       "metadata" => {
         "name" => "unittests.stable.example.io",
         "annotations" => {
-          rollout_conditions_annotation_key => "true",
+          rollout_conditions_annotation_key => rollout_conditions,
         },
       },
     ))
@@ -176,12 +185,12 @@ def test_instance_timeout_messages_with_rollout_conditions
         },
       })
 
-    cr.expects(:current_generation).returns(1)
-    cr.expects(:observed_generation).returns(1)
+    cr.expects(:current_generation).at_least_once.returns(1)
+    cr.expects(:observed_generation).at_least_once.returns(1)
     assert_equal(cr.timeout_message, Krane::KubernetesResource::STANDARD_TIMEOUT_MESSAGE)
 
-    cr.expects(:current_generation).returns(1)
-    cr.expects(:observed_generation).returns(2)
+    cr.expects(:current_generation).at_least_once.returns(1)
+    cr.expects(:observed_generation).at_least_once.returns(2)
     assert_equal(cr.timeout_message, Krane::CustomResource::TIMEOUT_MESSAGE_DIFFERENT_GENERATIONS)
   end