From 29e055c32652105a933a1fa8ce9b82be28121efd Mon Sep 17 00:00:00 2001
From: ayana-s <ayana.siddiqui@shopify.com>
Date: Thu, 3 Aug 2023 09:23:26 -0400
Subject: [PATCH] Squash

---
 lib/krane/kubernetes_resource/stateful_set.rb | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/lib/krane/kubernetes_resource/stateful_set.rb b/lib/krane/kubernetes_resource/stateful_set.rb
index ff579e768..4f06d7029 100644
--- a/lib/krane/kubernetes_resource/stateful_set.rb
+++ b/lib/krane/kubernetes_resource/stateful_set.rb
@@ -3,6 +3,7 @@
 module Krane
   class StatefulSet < PodSetBase
     TIMEOUT = 10.minutes
+    ONDELETE = 'OnDelete'
     SYNC_DEPENDENCIES = %w(Pod)
     REQUIRED_ROLLOUT_TYPES = %w(full).freeze
     attr_reader :pods
@@ -30,7 +31,7 @@ def deploy_succeeded?
         success &= desired_replicas == status_data['readyReplicas'].to_i
         success &= desired_replicas == status_data['currentReplicas'].to_i
 
-      elsif update_strategy == 'OnDelete'
+      elsif update_strategy == ONDELETE
         unless @success_assumption_warning_shown
           @logger.warn("WARNING: Your StatefulSet's updateStrategy is set to OnDelete, "\
                        "which means the deployment won't succeed until all pods are updated by deletion.")
@@ -47,7 +48,7 @@ def deploy_succeeded?
     end
 
     def deploy_failed?
-      return false if update_strategy == 'OnDelete' && required_rollout != 'full'
+      return false if update_strategy == ONDELETE && required_rollout != 'full'
       pods.present? && pods.any?(&:deploy_failed?) &&
       observed_generation == current_generation
     end