From ed7c13e63b03b2f32b68132ae1e9f5096dd72315 Mon Sep 17 00:00:00 2001
From: Hongchao Deng <hongchaodeng1@gmail.com>
Date: Fri, 5 May 2017 22:01:32 -0700
Subject: [PATCH] backup: change deployment update strategy to Recreate

---
 CHANGELOG.md               | 4 ++++
 pkg/util/k8sutil/backup.go | 3 +++
 2 files changed, 7 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index cd38ccb..44fc7fd 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -6,10 +6,14 @@
 
 ### Changed
 
+- Backup sidecar deployment created with `Recreate` strategy.
+
 ### Removed
 
 ### Fixed
 
+- [GH-1068] Backup sidecar deployment stuck at upgrading
+
 ### Deprecated
 
 ### Security
diff --git a/pkg/util/k8sutil/backup.go b/pkg/util/k8sutil/backup.go
index 209dc2c..7c3bca0 100644
--- a/pkg/util/k8sutil/backup.go
+++ b/pkg/util/k8sutil/backup.go
@@ -213,6 +213,9 @@ func NewBackupDeploymentManifest(name string, dplSel map[string]string, pl v1.Po
 		Spec: appsv1beta1.DeploymentSpec{
 			Selector: &metav1.LabelSelector{MatchLabels: pl.ObjectMeta.Labels},
 			Template: pl,
+			Strategy: appsv1beta1.DeploymentStrategy{
+				Type: appsv1beta1.RecreateDeploymentStrategyType,
+			},
 		},
 	}
 	addOwnerRefToObject(d.GetObjectMeta(), owner)