deployment_test.go 56 KB

1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586878889909192939495969798991001011021031041051061071081091101111121131141151161171181191201211221231241251261271281291301311321331341351361371381391401411421431441451461471481491501511521531541551561571581591601611621631641651661671681691701711721731741751761771781791801811821831841851861871881891901911921931941951961971981992002012022032042052062072082092102112122132142152162172182192202212222232242252262272282292302312322332342352362372382392402412422432442452462472482492502512522532542552562572582592602612622632642652662672682692702712722732742752762772782792802812822832842852862872882892902912922932942952962972982993003013023033043053063073083093103113123133143153163173183193203213223233243253263273283293303313323333343353363373383393403413423433443453463473483493503513523533543553563573583593603613623633643653663673683693703713723733743753763773783793803813823833843853863873883893903913923933943953963973983994004014024034044054064074084094104114124134144154164174184194204214224234244254264274284294304314324334344354364374384394404414424434444454464474484494504514524534544554564574584594604614624634644654664674684694704714724734744754764774784794804814824834844854864874884894904914924934944954964974984995005015025035045055065075085095105115125135145155165175185195205215225235245255265275285295305315325335345355365375385395405415425435445455465475485495505515525535545555565575585595605615625635645655665675685695705715725735745755765775785795805815825835845855865875885895905915925935945955965975985996006016026036046056066076086096106116126136146156166176186196206216226236246256266276286296306316326336346356366376386396406416426436446456466476486496506516526536546556566576586596606616626636646656666676686696706716726736746756766776786796806816826836846856866876886896906916926936946956966976986997007017027037047057067077087097107117127137147157167177187197207217227237247257267277287297307317327337347357367377387397407417427437447457467477487497507517527537547557567577587597607617627637647657667677687697707717727737747757767777787797807817827837847857867877887897907917927937947957967977987998008018028038048058068078088098108118128138148158168178188198208218228238248258268278288298308318328338348358368378388398408418428438448458468478488498508518528538548558568578588598608618628638648658668678688698708718728738748758768778788798808818828838848858868878888898908918928938948958968978988999009019029039049059069079089099109119129139149159169179189199209219229239249259269279289299309319329339349359369379389399409419429439449459469479489499509519529539549559569579589599609619629639649659669679689699709719729739749759769779789799809819829839849859869879889899909919929939949959969979989991000100110021003100410051006100710081009101010111012101310141015101610171018101910201021102210231024102510261027102810291030103110321033103410351036103710381039104010411042104310441045104610471048104910501051105210531054105510561057105810591060106110621063106410651066106710681069107010711072107310741075107610771078107910801081108210831084108510861087108810891090109110921093109410951096109710981099110011011102110311041105110611071108110911101111111211131114111511161117111811191120112111221123112411251126112711281129113011311132113311341135113611371138113911401141114211431144114511461147114811491150115111521153115411551156115711581159116011611162116311641165116611671168116911701171117211731174117511761177117811791180118111821183118411851186118711881189119011911192119311941195119611971198119912001201120212031204120512061207120812091210121112121213121412151216121712181219122012211222122312241225122612271228122912301231123212331234123512361237123812391240124112421243124412451246124712481249125012511252125312541255125612571258125912601261126212631264126512661267126812691270127112721273127412751276127712781279128012811282128312841285128612871288128912901291129212931294129512961297129812991300130113021303130413051306130713081309131013111312131313141315131613171318131913201321132213231324132513261327132813291330133113321333133413351336133713381339134013411342134313441345134613471348134913501351135213531354135513561357135813591360136113621363136413651366136713681369137013711372137313741375137613771378137913801381138213831384138513861387138813891390139113921393139413951396139713981399140014011402140314041405140614071408140914101411141214131414141514161417141814191420142114221423142414251426142714281429143014311432143314341435143614371438143914401441144214431444144514461447144814491450145114521453145414551456145714581459146014611462
  1. /*
  2. Copyright 2017 The Kubernetes Authors.
  3. Licensed under the Apache License, Version 2.0 (the "License");
  4. you may not use this file except in compliance with the License.
  5. You may obtain a copy of the License at
  6. http://www.apache.org/licenses/LICENSE-2.0
  7. Unless required by applicable law or agreed to in writing, software
  8. distributed under the License is distributed on an "AS IS" BASIS,
  9. WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
  10. See the License for the specific language governing permissions and
  11. limitations under the License.
  12. */
  13. package deployment
  14. import (
  15. "fmt"
  16. "reflect"
  17. "strings"
  18. "testing"
  19. apps "k8s.io/api/apps/v1"
  20. "k8s.io/api/core/v1"
  21. metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
  22. "k8s.io/apimachinery/pkg/util/intstr"
  23. "k8s.io/apimachinery/pkg/util/uuid"
  24. "k8s.io/apimachinery/pkg/util/wait"
  25. "k8s.io/client-go/util/retry"
  26. deploymentutil "k8s.io/kubernetes/pkg/controller/deployment/util"
  27. "k8s.io/kubernetes/test/integration/framework"
  28. "k8s.io/utils/pointer"
  29. )
  30. func TestNewDeployment(t *testing.T) {
  31. s, closeFn, rm, dc, informers, c := dcSetup(t)
  32. defer closeFn()
  33. name := "test-new-deployment"
  34. ns := framework.CreateTestingNamespace(name, s, t)
  35. defer framework.DeleteTestingNamespace(ns, s, t)
  36. replicas := int32(20)
  37. tester := &deploymentTester{t: t, c: c, deployment: newDeployment(name, ns.Name, replicas)}
  38. tester.deployment.Spec.MinReadySeconds = 4
  39. tester.deployment.Annotations = map[string]string{"test": "should-copy-to-replica-set", v1.LastAppliedConfigAnnotation: "should-not-copy-to-replica-set"}
  40. var err error
  41. tester.deployment, err = c.AppsV1().Deployments(ns.Name).Create(tester.deployment)
  42. if err != nil {
  43. t.Fatalf("failed to create deployment %s: %v", tester.deployment.Name, err)
  44. }
  45. // Start informer and controllers
  46. stopCh := make(chan struct{})
  47. defer close(stopCh)
  48. informers.Start(stopCh)
  49. go rm.Run(5, stopCh)
  50. go dc.Run(5, stopCh)
  51. // Wait for the Deployment to be updated to revision 1
  52. if err := tester.waitForDeploymentRevisionAndImage("1", fakeImage); err != nil {
  53. t.Fatal(err)
  54. }
  55. // Make sure the Deployment completes while manually marking Deployment pods as ready at the same time.
  56. // Use soft check because this deployment was just created and rolling update strategy might be violated.
  57. if err := tester.waitForDeploymentCompleteAndMarkPodsReady(); err != nil {
  58. t.Fatal(err)
  59. }
  60. // Check new RS annotations
  61. newRS, err := tester.expectNewReplicaSet()
  62. if err != nil {
  63. t.Fatal(err)
  64. }
  65. if newRS.Annotations["test"] != "should-copy-to-replica-set" {
  66. t.Errorf("expected new ReplicaSet annotations copied from Deployment %s, got: %v", tester.deployment.Name, newRS.Annotations)
  67. }
  68. if newRS.Annotations[v1.LastAppliedConfigAnnotation] != "" {
  69. t.Errorf("expected new ReplicaSet last-applied annotation not copied from Deployment %s", tester.deployment.Name)
  70. }
  71. // New RS should contain pod-template-hash in its selector, label, and template label
  72. rsHash, err := checkRSHashLabels(newRS)
  73. if err != nil {
  74. t.Error(err)
  75. }
  76. // All pods targeted by the deployment should contain pod-template-hash in their labels
  77. selector, err := metav1.LabelSelectorAsSelector(tester.deployment.Spec.Selector)
  78. if err != nil {
  79. t.Fatalf("failed to parse deployment %s selector: %v", name, err)
  80. }
  81. pods, err := c.CoreV1().Pods(ns.Name).List(metav1.ListOptions{LabelSelector: selector.String()})
  82. if err != nil {
  83. t.Fatalf("failed to list pods of deployment %s: %v", name, err)
  84. }
  85. if len(pods.Items) != int(replicas) {
  86. t.Errorf("expected %d pods, got %d pods", replicas, len(pods.Items))
  87. }
  88. podHash, err := checkPodsHashLabel(pods)
  89. if err != nil {
  90. t.Error(err)
  91. }
  92. if rsHash != podHash {
  93. t.Errorf("found mismatching pod-template-hash value: rs hash = %s whereas pod hash = %s", rsHash, podHash)
  94. }
  95. }
  96. // Deployments should support roll out, roll back, and roll over.
  97. // TODO: drop the rollback portions of this test when extensions/v1beta1 is no longer served
  98. // and rollback endpoint is no longer supported.
  99. func TestDeploymentRollingUpdate(t *testing.T) {
  100. s, closeFn, rm, dc, informers, c := dcSetup(t)
  101. defer closeFn()
  102. name := "test-rolling-update-deployment"
  103. ns := framework.CreateTestingNamespace(name, s, t)
  104. defer framework.DeleteTestingNamespace(ns, s, t)
  105. // Start informer and controllers
  106. stopCh := make(chan struct{})
  107. defer close(stopCh)
  108. informers.Start(stopCh)
  109. go rm.Run(5, stopCh)
  110. go dc.Run(5, stopCh)
  111. replicas := int32(20)
  112. tester := &deploymentTester{t: t, c: c, deployment: newDeployment(name, ns.Name, replicas)}
  113. tester.deployment.Spec.MinReadySeconds = 4
  114. quarter := intstr.FromString("25%")
  115. tester.deployment.Spec.Strategy.RollingUpdate = &apps.RollingUpdateDeployment{
  116. MaxUnavailable: &quarter,
  117. MaxSurge: &quarter,
  118. }
  119. // Create a deployment.
  120. var err error
  121. tester.deployment, err = c.AppsV1().Deployments(ns.Name).Create(tester.deployment)
  122. if err != nil {
  123. t.Fatalf("failed to create deployment %s: %v", tester.deployment.Name, err)
  124. }
  125. oriImage := tester.deployment.Spec.Template.Spec.Containers[0].Image
  126. if err := tester.waitForDeploymentRevisionAndImage("1", oriImage); err != nil {
  127. t.Fatal(err)
  128. }
  129. if err := tester.waitForDeploymentCompleteAndMarkPodsReady(); err != nil {
  130. t.Fatal(err)
  131. }
  132. // 1. Roll out a new image.
  133. image := "new-image"
  134. if oriImage == image {
  135. t.Fatalf("bad test setup, deployment %s roll out with the same image", tester.deployment.Name)
  136. }
  137. imageFn := func(update *apps.Deployment) {
  138. update.Spec.Template.Spec.Containers[0].Image = image
  139. }
  140. tester.deployment, err = tester.updateDeployment(imageFn)
  141. if err != nil {
  142. t.Fatalf("failed to update deployment %s: %v", tester.deployment.Name, err)
  143. }
  144. if err := tester.waitForDeploymentRevisionAndImage("2", image); err != nil {
  145. t.Fatal(err)
  146. }
  147. if err := tester.waitForDeploymentCompleteAndCheckRollingAndMarkPodsReady(); err != nil {
  148. t.Fatal(err)
  149. }
  150. // 2. Roll back to the last revision.
  151. revision := int64(0)
  152. rollback := newDeploymentRollback(tester.deployment.Name, nil, revision)
  153. if err = c.ExtensionsV1beta1().Deployments(ns.Name).Rollback(rollback); err != nil {
  154. t.Fatalf("failed to roll back deployment %s to last revision: %v", tester.deployment.Name, err)
  155. }
  156. // Wait for the deployment to start rolling back
  157. if err = tester.waitForDeploymentRollbackCleared(); err != nil {
  158. t.Fatalf("failed to roll back deployment %s to last revision: %v", tester.deployment.Name, err)
  159. }
  160. // Wait for the deployment to be rolled back to the template stored in revision 1 and rolled forward to revision 3.
  161. if err := tester.waitForDeploymentRevisionAndImage("3", oriImage); err != nil {
  162. t.Fatal(err)
  163. }
  164. if err := tester.waitForDeploymentCompleteAndCheckRollingAndMarkPodsReady(); err != nil {
  165. t.Fatal(err)
  166. }
  167. // 3. Roll over a deployment before the previous rolling update finishes.
  168. image = "dont-finish"
  169. imageFn = func(update *apps.Deployment) {
  170. update.Spec.Template.Spec.Containers[0].Image = image
  171. }
  172. tester.deployment, err = tester.updateDeployment(imageFn)
  173. if err != nil {
  174. t.Fatalf("failed to update deployment %s: %v", tester.deployment.Name, err)
  175. }
  176. if err := tester.waitForDeploymentRevisionAndImage("4", image); err != nil {
  177. t.Fatal(err)
  178. }
  179. // We don't mark pods as ready so that rollout won't finish.
  180. // Before the rollout finishes, trigger another rollout.
  181. image = "rollover"
  182. imageFn = func(update *apps.Deployment) {
  183. update.Spec.Template.Spec.Containers[0].Image = image
  184. }
  185. tester.deployment, err = tester.updateDeployment(imageFn)
  186. if err != nil {
  187. t.Fatalf("failed to update deployment %s: %v", tester.deployment.Name, err)
  188. }
  189. if err := tester.waitForDeploymentRevisionAndImage("5", image); err != nil {
  190. t.Fatal(err)
  191. }
  192. if err := tester.waitForDeploymentCompleteAndCheckRollingAndMarkPodsReady(); err != nil {
  193. t.Fatal(err)
  194. }
  195. _, allOldRSs, err := deploymentutil.GetOldReplicaSets(tester.deployment, c.AppsV1())
  196. if err != nil {
  197. t.Fatalf("failed retrieving old replicasets of deployment %s: %v", tester.deployment.Name, err)
  198. }
  199. for _, oldRS := range allOldRSs {
  200. if *oldRS.Spec.Replicas != 0 {
  201. t.Errorf("expected old replicaset %s of deployment %s to have 0 replica, got %d", oldRS.Name, tester.deployment.Name, *oldRS.Spec.Replicas)
  202. }
  203. }
  204. }
  205. // selectors are IMMUTABLE for all API versions except apps/v1beta1 and extensions/v1beta1
  206. func TestDeploymentSelectorImmutability(t *testing.T) {
  207. s, closeFn, c := dcSimpleSetup(t)
  208. defer closeFn()
  209. name := "test-deployment-selector-immutability"
  210. ns := framework.CreateTestingNamespace(name, s, t)
  211. defer framework.DeleteTestingNamespace(ns, s, t)
  212. tester := &deploymentTester{t: t, c: c, deployment: newDeployment(name, ns.Name, int32(20))}
  213. var err error
  214. tester.deployment, err = c.AppsV1().Deployments(ns.Name).Create(tester.deployment)
  215. if err != nil {
  216. t.Fatalf("failed to create apps/v1 deployment %s: %v", tester.deployment.Name, err)
  217. }
  218. // test to ensure extensions/v1beta1 selector is mutable
  219. // TODO: drop the extensions/v1beta1 portion of this test when extensions/v1beta1 is no longer served
  220. newSelectorLabels := map[string]string{"name_extensions_v1beta1": "test_extensions_v1beta1"}
  221. deploymentExtensionsV1beta1, err := c.ExtensionsV1beta1().Deployments(ns.Name).Get(name, metav1.GetOptions{})
  222. if err != nil {
  223. t.Fatalf("failed to get extensions/v1beta deployment %s: %v", name, err)
  224. }
  225. deploymentExtensionsV1beta1.Spec.Selector.MatchLabels = newSelectorLabels
  226. deploymentExtensionsV1beta1.Spec.Template.Labels = newSelectorLabels
  227. updatedDeploymentExtensionsV1beta1, err := c.ExtensionsV1beta1().Deployments(ns.Name).Update(deploymentExtensionsV1beta1)
  228. if err != nil {
  229. t.Fatalf("failed to update extensions/v1beta1 deployment %s: %v", deploymentExtensionsV1beta1.Name, err)
  230. }
  231. if !reflect.DeepEqual(updatedDeploymentExtensionsV1beta1.Spec.Selector.MatchLabels, newSelectorLabels) {
  232. t.Errorf("selector should be changed for extensions/v1beta1, expected: %v, got: %v", newSelectorLabels, updatedDeploymentExtensionsV1beta1.Spec.Selector.MatchLabels)
  233. }
  234. // test to ensure apps/v1beta1 selector is mutable
  235. deploymentAppsV1beta1, err := c.AppsV1beta1().Deployments(ns.Name).Get(updatedDeploymentExtensionsV1beta1.Name, metav1.GetOptions{})
  236. if err != nil {
  237. t.Fatalf("failed to get apps/v1beta1 deployment %s: %v", updatedDeploymentExtensionsV1beta1.Name, err)
  238. }
  239. newSelectorLabels = map[string]string{"name_apps_v1beta1": "test_apps_v1beta1"}
  240. deploymentAppsV1beta1.Spec.Selector.MatchLabels = newSelectorLabels
  241. deploymentAppsV1beta1.Spec.Template.Labels = newSelectorLabels
  242. updatedDeploymentAppsV1beta1, err := c.AppsV1beta1().Deployments(ns.Name).Update(deploymentAppsV1beta1)
  243. if err != nil {
  244. t.Fatalf("failed to update apps/v1beta1 deployment %s: %v", deploymentAppsV1beta1.Name, err)
  245. }
  246. if !reflect.DeepEqual(updatedDeploymentAppsV1beta1.Spec.Selector.MatchLabels, newSelectorLabels) {
  247. t.Errorf("selector should be changed for apps/v1beta1, expected: %v, got: %v", newSelectorLabels, updatedDeploymentAppsV1beta1.Spec.Selector.MatchLabels)
  248. }
  249. // test to ensure apps/v1beta2 selector is immutable
  250. deploymentAppsV1beta2, err := c.AppsV1beta2().Deployments(ns.Name).Get(updatedDeploymentAppsV1beta1.Name, metav1.GetOptions{})
  251. if err != nil {
  252. t.Fatalf("failed to get apps/v1beta2 deployment %s: %v", updatedDeploymentAppsV1beta1.Name, err)
  253. }
  254. newSelectorLabels = map[string]string{"name_apps_v1beta2": "test_apps_v1beta2"}
  255. deploymentAppsV1beta2.Spec.Selector.MatchLabels = newSelectorLabels
  256. deploymentAppsV1beta2.Spec.Template.Labels = newSelectorLabels
  257. _, err = c.AppsV1beta2().Deployments(ns.Name).Update(deploymentAppsV1beta2)
  258. if err == nil {
  259. t.Fatalf("failed to provide validation error when changing immutable selector when updating apps/v1beta2 deployment %s", deploymentAppsV1beta2.Name)
  260. }
  261. expectedErrType := "Invalid value"
  262. expectedErrDetail := "field is immutable"
  263. if !strings.Contains(err.Error(), expectedErrType) || !strings.Contains(err.Error(), expectedErrDetail) {
  264. t.Errorf("error message does not match, expected type: %s, expected detail: %s, got: %s", expectedErrType, expectedErrDetail, err.Error())
  265. }
  266. // test to ensure apps/v1 selector is immutable
  267. deploymentAppsV1, err := c.AppsV1().Deployments(ns.Name).Get(updatedDeploymentAppsV1beta1.Name, metav1.GetOptions{})
  268. if err != nil {
  269. t.Fatalf("failed to get apps/v1 deployment %s: %v", updatedDeploymentAppsV1beta1.Name, err)
  270. }
  271. newSelectorLabels = map[string]string{"name_apps_v1": "test_apps_v1"}
  272. deploymentAppsV1.Spec.Selector.MatchLabels = newSelectorLabels
  273. deploymentAppsV1.Spec.Template.Labels = newSelectorLabels
  274. _, err = c.AppsV1().Deployments(ns.Name).Update(deploymentAppsV1)
  275. if err == nil {
  276. t.Fatalf("failed to provide validation error when changing immutable selector when updating apps/v1 deployment %s", deploymentAppsV1.Name)
  277. }
  278. if !strings.Contains(err.Error(), expectedErrType) || !strings.Contains(err.Error(), expectedErrDetail) {
  279. t.Errorf("error message does not match, expected type: %s, expected detail: %s, got: %s", expectedErrType, expectedErrDetail, err.Error())
  280. }
  281. }
  282. // Paused deployment should not start new rollout
  283. func TestPausedDeployment(t *testing.T) {
  284. s, closeFn, rm, dc, informers, c := dcSetup(t)
  285. defer closeFn()
  286. name := "test-paused-deployment"
  287. ns := framework.CreateTestingNamespace(name, s, t)
  288. defer framework.DeleteTestingNamespace(ns, s, t)
  289. replicas := int32(1)
  290. tester := &deploymentTester{t: t, c: c, deployment: newDeployment(name, ns.Name, replicas)}
  291. tester.deployment.Spec.Paused = true
  292. tgps := int64(1)
  293. tester.deployment.Spec.Template.Spec.TerminationGracePeriodSeconds = &tgps
  294. var err error
  295. tester.deployment, err = c.AppsV1().Deployments(ns.Name).Create(tester.deployment)
  296. if err != nil {
  297. t.Fatalf("failed to create deployment %s: %v", tester.deployment.Name, err)
  298. }
  299. // Start informer and controllers
  300. stopCh := make(chan struct{})
  301. defer close(stopCh)
  302. informers.Start(stopCh)
  303. go rm.Run(5, stopCh)
  304. go dc.Run(5, stopCh)
  305. // Verify that the paused deployment won't create new replica set.
  306. if err := tester.expectNoNewReplicaSet(); err != nil {
  307. t.Fatal(err)
  308. }
  309. // Resume the deployment
  310. tester.deployment, err = tester.updateDeployment(resumeFn)
  311. if err != nil {
  312. t.Fatalf("failed to resume deployment %s: %v", tester.deployment.Name, err)
  313. }
  314. // Wait for the controller to notice the resume.
  315. if err := tester.waitForObservedDeployment(tester.deployment.Generation); err != nil {
  316. t.Fatal(err)
  317. }
  318. // Wait for the Deployment to be updated to revision 1
  319. if err := tester.waitForDeploymentRevisionAndImage("1", fakeImage); err != nil {
  320. t.Fatal(err)
  321. }
  322. // Make sure the Deployment completes while manually marking Deployment pods as ready at the same time.
  323. // Use soft check because this deployment was just created and rolling update strategy might be violated.
  324. if err := tester.waitForDeploymentCompleteAndMarkPodsReady(); err != nil {
  325. t.Fatal(err)
  326. }
  327. // A new replicaset should be created.
  328. if _, err := tester.expectNewReplicaSet(); err != nil {
  329. t.Fatal(err)
  330. }
  331. // Pause the deployment.
  332. // The paused deployment shouldn't trigger a new rollout.
  333. tester.deployment, err = tester.updateDeployment(pauseFn)
  334. if err != nil {
  335. t.Fatalf("failed to pause deployment %s: %v", tester.deployment.Name, err)
  336. }
  337. // Wait for the controller to notice the pause.
  338. if err := tester.waitForObservedDeployment(tester.deployment.Generation); err != nil {
  339. t.Fatal(err)
  340. }
  341. // Update the deployment template
  342. newTGPS := int64(0)
  343. tester.deployment, err = tester.updateDeployment(func(update *apps.Deployment) {
  344. update.Spec.Template.Spec.TerminationGracePeriodSeconds = &newTGPS
  345. })
  346. if err != nil {
  347. t.Fatalf("failed updating template of deployment %s: %v", tester.deployment.Name, err)
  348. }
  349. // Wait for the controller to notice the rollout.
  350. if err := tester.waitForObservedDeployment(tester.deployment.Generation); err != nil {
  351. t.Fatal(err)
  352. }
  353. // Verify that the paused deployment won't create new replica set.
  354. if err := tester.expectNoNewReplicaSet(); err != nil {
  355. t.Fatal(err)
  356. }
  357. _, allOldRs, err := deploymentutil.GetOldReplicaSets(tester.deployment, c.AppsV1())
  358. if err != nil {
  359. t.Fatalf("failed retrieving old replicasets of deployment %s: %v", tester.deployment.Name, err)
  360. }
  361. if len(allOldRs) != 1 {
  362. t.Errorf("expected an old replica set, got %v", allOldRs)
  363. }
  364. if *allOldRs[0].Spec.Template.Spec.TerminationGracePeriodSeconds == newTGPS {
  365. t.Errorf("TerminationGracePeriodSeconds on the replica set should be %d, got %d", tgps, newTGPS)
  366. }
  367. }
  368. // Paused deployment can be scaled
  369. func TestScalePausedDeployment(t *testing.T) {
  370. s, closeFn, rm, dc, informers, c := dcSetup(t)
  371. defer closeFn()
  372. name := "test-scale-paused-deployment"
  373. ns := framework.CreateTestingNamespace(name, s, t)
  374. defer framework.DeleteTestingNamespace(ns, s, t)
  375. replicas := int32(1)
  376. tester := &deploymentTester{t: t, c: c, deployment: newDeployment(name, ns.Name, replicas)}
  377. tgps := int64(1)
  378. tester.deployment.Spec.Template.Spec.TerminationGracePeriodSeconds = &tgps
  379. var err error
  380. tester.deployment, err = c.AppsV1().Deployments(ns.Name).Create(tester.deployment)
  381. if err != nil {
  382. t.Fatalf("failed to create deployment %s: %v", tester.deployment.Name, err)
  383. }
  384. // Start informer and controllers
  385. stopCh := make(chan struct{})
  386. defer close(stopCh)
  387. informers.Start(stopCh)
  388. go rm.Run(5, stopCh)
  389. go dc.Run(5, stopCh)
  390. // Wait for the Deployment to be updated to revision 1
  391. if err := tester.waitForDeploymentRevisionAndImage("1", fakeImage); err != nil {
  392. t.Fatal(err)
  393. }
  394. // Make sure the Deployment completes while manually marking Deployment pods as ready at the same time.
  395. // Use soft check because this deployment was just created and rolling update strategy might be violated.
  396. if err := tester.waitForDeploymentCompleteAndMarkPodsReady(); err != nil {
  397. t.Fatal(err)
  398. }
  399. // A new replicaset should be created.
  400. if _, err := tester.expectNewReplicaSet(); err != nil {
  401. t.Fatal(err)
  402. }
  403. // Pause the deployment.
  404. tester.deployment, err = tester.updateDeployment(pauseFn)
  405. if err != nil {
  406. t.Fatalf("failed to pause deployment %s: %v", tester.deployment.Name, err)
  407. }
  408. // Wait for the controller to notice the scale.
  409. if err := tester.waitForObservedDeployment(tester.deployment.Generation); err != nil {
  410. t.Fatal(err)
  411. }
  412. // Scale the paused deployment.
  413. newReplicas := int32(10)
  414. tester.deployment, err = tester.updateDeployment(func(update *apps.Deployment) {
  415. update.Spec.Replicas = &newReplicas
  416. })
  417. if err != nil {
  418. t.Fatalf("failed updating deployment %s: %v", tester.deployment.Name, err)
  419. }
  420. // Wait for the controller to notice the scale.
  421. if err := tester.waitForObservedDeployment(tester.deployment.Generation); err != nil {
  422. t.Fatal(err)
  423. }
  424. // Verify that the new replicaset is scaled.
  425. rs, err := tester.expectNewReplicaSet()
  426. if err != nil {
  427. t.Fatal(err)
  428. }
  429. if *rs.Spec.Replicas != newReplicas {
  430. t.Errorf("expected new replicaset replicas = %d, got %d", newReplicas, *rs.Spec.Replicas)
  431. }
  432. // Make sure the Deployment completes while manually marking Deployment pods as ready at the same time.
  433. // Use soft check because this deployment was just scaled and rolling update strategy might be violated.
  434. if err := tester.waitForDeploymentCompleteAndMarkPodsReady(); err != nil {
  435. t.Fatal(err)
  436. }
  437. }
  438. // Deployment rollout shouldn't be blocked on hash collisions
  439. func TestDeploymentHashCollision(t *testing.T) {
  440. s, closeFn, rm, dc, informers, c := dcSetup(t)
  441. defer closeFn()
  442. name := "test-hash-collision-deployment"
  443. ns := framework.CreateTestingNamespace(name, s, t)
  444. defer framework.DeleteTestingNamespace(ns, s, t)
  445. replicas := int32(1)
  446. tester := &deploymentTester{t: t, c: c, deployment: newDeployment(name, ns.Name, replicas)}
  447. var err error
  448. tester.deployment, err = c.AppsV1().Deployments(ns.Name).Create(tester.deployment)
  449. if err != nil {
  450. t.Fatalf("failed to create deployment %s: %v", tester.deployment.Name, err)
  451. }
  452. // Start informer and controllers
  453. stopCh := make(chan struct{})
  454. defer close(stopCh)
  455. informers.Start(stopCh)
  456. go rm.Run(5, stopCh)
  457. go dc.Run(5, stopCh)
  458. // Wait for the Deployment to be updated to revision 1
  459. if err := tester.waitForDeploymentRevisionAndImage("1", fakeImage); err != nil {
  460. t.Fatal(err)
  461. }
  462. // Mock a hash collision
  463. newRS, err := deploymentutil.GetNewReplicaSet(tester.deployment, c.AppsV1())
  464. if err != nil {
  465. t.Fatalf("failed getting new replicaset of deployment %s: %v", tester.deployment.Name, err)
  466. }
  467. if newRS == nil {
  468. t.Fatalf("unable to find new replicaset of deployment %s", tester.deployment.Name)
  469. }
  470. _, err = tester.updateReplicaSet(newRS.Name, func(update *apps.ReplicaSet) {
  471. *update.Spec.Template.Spec.TerminationGracePeriodSeconds = int64(5)
  472. })
  473. if err != nil {
  474. t.Fatalf("failed updating replicaset %s template: %v", newRS.Name, err)
  475. }
  476. // Expect deployment collision counter to increment
  477. if err := wait.PollImmediate(pollInterval, pollTimeout, func() (bool, error) {
  478. d, err := c.AppsV1().Deployments(ns.Name).Get(tester.deployment.Name, metav1.GetOptions{})
  479. if err != nil {
  480. return false, nil
  481. }
  482. return d.Status.CollisionCount != nil && *d.Status.CollisionCount == int32(1), nil
  483. }); err != nil {
  484. t.Fatalf("Failed to increment collision counter for deployment %q: %v", tester.deployment.Name, err)
  485. }
  486. // Expect a new ReplicaSet to be created
  487. if err := tester.waitForDeploymentRevisionAndImage("2", fakeImage); err != nil {
  488. t.Fatal(err)
  489. }
  490. }
  491. // Deployment supports rollback even when there's old replica set without revision.
  492. // TODO: drop this test when extensions/v1beta1 is no longer served
  493. func TestRollbackDeploymentRSNoRevision(t *testing.T) {
  494. s, closeFn, rm, dc, informers, c := dcSetup(t)
  495. defer closeFn()
  496. name := "test-rollback-no-revision-deployment"
  497. ns := framework.CreateTestingNamespace(name, s, t)
  498. defer framework.DeleteTestingNamespace(ns, s, t)
  499. // Create an old RS without revision
  500. rsName := "test-rollback-no-revision-controller"
  501. rsReplicas := int32(1)
  502. rs := newReplicaSet(rsName, ns.Name, rsReplicas)
  503. rs.Annotations = make(map[string]string)
  504. rs.Annotations["make"] = "difference"
  505. rs.Spec.Template.Spec.Containers[0].Image = "different-image"
  506. _, err := c.AppsV1().ReplicaSets(ns.Name).Create(rs)
  507. if err != nil {
  508. t.Fatalf("failed to create replicaset %s: %v", rsName, err)
  509. }
  510. replicas := int32(1)
  511. tester := &deploymentTester{t: t, c: c, deployment: newDeployment(name, ns.Name, replicas)}
  512. oriImage := tester.deployment.Spec.Template.Spec.Containers[0].Image
  513. // Set absolute rollout limits (defaults changed to percentages)
  514. max := intstr.FromInt(1)
  515. tester.deployment.Spec.Strategy.RollingUpdate.MaxUnavailable = &max
  516. tester.deployment.Spec.Strategy.RollingUpdate.MaxSurge = &max
  517. // Create a deployment which have different template than the replica set created above.
  518. if tester.deployment, err = c.AppsV1().Deployments(ns.Name).Create(tester.deployment); err != nil {
  519. t.Fatalf("failed to create deployment %s: %v", tester.deployment.Name, err)
  520. }
  521. // Start informer and controllers
  522. stopCh := make(chan struct{})
  523. defer close(stopCh)
  524. informers.Start(stopCh)
  525. go rm.Run(5, stopCh)
  526. go dc.Run(5, stopCh)
  527. // Wait for the Deployment to be updated to revision 1
  528. if err = tester.waitForDeploymentRevisionAndImage("1", fakeImage); err != nil {
  529. t.Fatal(err)
  530. }
  531. // 1. Rollback to the last revision
  532. // Since there's only 1 revision in history, it should still be revision 1
  533. revision := int64(0)
  534. rollback := newDeploymentRollback(tester.deployment.Name, nil, revision)
  535. if err = c.ExtensionsV1beta1().Deployments(ns.Name).Rollback(rollback); err != nil {
  536. t.Fatalf("failed to roll back deployment %s to last revision: %v", tester.deployment.Name, err)
  537. }
  538. // Wait for the deployment to start rolling back
  539. if err = tester.waitForDeploymentRollbackCleared(); err != nil {
  540. t.Fatalf("failed to roll back deployment %s to last revision: %v", tester.deployment.Name, err)
  541. }
  542. // TODO: report RollbackRevisionNotFound in deployment status and check it here
  543. // The pod template shouldn't change since there's no last revision
  544. // Check if the deployment is still revision 1 and still has the old pod template
  545. err = tester.checkDeploymentRevisionAndImage("1", oriImage)
  546. if err != nil {
  547. t.Fatal(err)
  548. }
  549. // 2. Update the deployment to revision 2.
  550. updatedImage := "update"
  551. tester.deployment, err = tester.updateDeployment(func(update *apps.Deployment) {
  552. update.Spec.Template.Spec.Containers[0].Name = updatedImage
  553. update.Spec.Template.Spec.Containers[0].Image = updatedImage
  554. })
  555. if err != nil {
  556. t.Fatalf("failed updating deployment %s: %v", tester.deployment.Name, err)
  557. }
  558. // Use observedGeneration to determine if the controller noticed the pod template update.
  559. // Wait for the controller to notice the resume.
  560. if err = tester.waitForObservedDeployment(tester.deployment.Generation); err != nil {
  561. t.Fatal(err)
  562. }
  563. // Wait for it to be updated to revision 2
  564. if err = tester.waitForDeploymentRevisionAndImage("2", updatedImage); err != nil {
  565. t.Fatal(err)
  566. }
  567. // Wait for the Deployment to complete while manually marking Deployment pods as ready at the same time
  568. if err = tester.waitForDeploymentCompleteAndCheckRollingAndMarkPodsReady(); err != nil {
  569. t.Fatal(err)
  570. }
  571. // 3. Update the deploymentRollback to rollback to revision 1
  572. revision = int64(1)
  573. rollback = newDeploymentRollback(tester.deployment.Name, nil, revision)
  574. if err = c.ExtensionsV1beta1().Deployments(ns.Name).Rollback(rollback); err != nil {
  575. t.Fatalf("failed to roll back deployment %s to revision %d: %v", tester.deployment.Name, revision, err)
  576. }
  577. // Wait for the deployment to start rolling back
  578. if err = tester.waitForDeploymentRollbackCleared(); err != nil {
  579. t.Fatalf("failed to roll back deployment %s to revision %d: %v", tester.deployment.Name, revision, err)
  580. }
  581. // TODO: report RollbackDone in deployment status and check it here
  582. // The pod template should be updated to the one in revision 1
  583. // Wait for it to be updated to revision 3
  584. if err = tester.waitForDeploymentRevisionAndImage("3", oriImage); err != nil {
  585. t.Fatal(err)
  586. }
  587. // Wait for the Deployment to complete while manually marking Deployment pods as ready at the same time
  588. if err = tester.waitForDeploymentCompleteAndCheckRollingAndMarkPodsReady(); err != nil {
  589. t.Fatal(err)
  590. }
  591. }
  592. func checkRSHashLabels(rs *apps.ReplicaSet) (string, error) {
  593. hash := rs.Labels[apps.DefaultDeploymentUniqueLabelKey]
  594. selectorHash := rs.Spec.Selector.MatchLabels[apps.DefaultDeploymentUniqueLabelKey]
  595. templateLabelHash := rs.Spec.Template.Labels[apps.DefaultDeploymentUniqueLabelKey]
  596. if hash != selectorHash || selectorHash != templateLabelHash {
  597. return "", fmt.Errorf("mismatching hash value found in replicaset %s: %#v", rs.Name, rs)
  598. }
  599. if len(hash) == 0 {
  600. return "", fmt.Errorf("unexpected replicaset %s missing required pod-template-hash labels", rs.Name)
  601. }
  602. if !strings.HasSuffix(rs.Name, hash) {
  603. return "", fmt.Errorf("unexpected replicaset %s name suffix doesn't match hash %s", rs.Name, hash)
  604. }
  605. return hash, nil
  606. }
  607. func checkPodsHashLabel(pods *v1.PodList) (string, error) {
  608. if len(pods.Items) == 0 {
  609. return "", fmt.Errorf("no pods given")
  610. }
  611. var hash string
  612. for _, pod := range pods.Items {
  613. podHash := pod.Labels[apps.DefaultDeploymentUniqueLabelKey]
  614. if len(podHash) == 0 {
  615. return "", fmt.Errorf("found pod %s missing pod-template-hash label: %#v", pod.Name, pods)
  616. }
  617. // Save the first valid hash
  618. if len(hash) == 0 {
  619. hash = podHash
  620. }
  621. if podHash != hash {
  622. return "", fmt.Errorf("found pod %s with mismatching pod-template-hash value %s: %#v", pod.Name, podHash, pods)
  623. }
  624. }
  625. return hash, nil
  626. }
  627. // Deployment should have a timeout condition when it fails to progress after given deadline.
  628. func TestFailedDeployment(t *testing.T) {
  629. s, closeFn, rm, dc, informers, c := dcSetup(t)
  630. defer closeFn()
  631. name := "test-failed-deployment"
  632. ns := framework.CreateTestingNamespace(name, s, t)
  633. defer framework.DeleteTestingNamespace(ns, s, t)
  634. deploymentName := "progress-check"
  635. replicas := int32(1)
  636. three := int32(3)
  637. tester := &deploymentTester{t: t, c: c, deployment: newDeployment(deploymentName, ns.Name, replicas)}
  638. tester.deployment.Spec.ProgressDeadlineSeconds = &three
  639. var err error
  640. tester.deployment, err = c.AppsV1().Deployments(ns.Name).Create(tester.deployment)
  641. if err != nil {
  642. t.Fatalf("failed to create deployment %q: %v", deploymentName, err)
  643. }
  644. // Start informer and controllers
  645. stopCh := make(chan struct{})
  646. defer close(stopCh)
  647. informers.Start(stopCh)
  648. go rm.Run(5, stopCh)
  649. go dc.Run(5, stopCh)
  650. if err = tester.waitForDeploymentUpdatedReplicasGTE(replicas); err != nil {
  651. t.Fatal(err)
  652. }
  653. // Pods are not marked as Ready, therefore the deployment progress will eventually timeout after progressDeadlineSeconds has passed.
  654. // Wait for the deployment to have a progress timeout condition.
  655. if err = tester.waitForDeploymentWithCondition(deploymentutil.TimedOutReason, apps.DeploymentProgressing); err != nil {
  656. t.Fatal(err)
  657. }
  658. // Manually mark pods as Ready and wait for deployment to complete.
  659. if err := tester.waitForDeploymentCompleteAndMarkPodsReady(); err != nil {
  660. t.Fatalf("deployment %q fails to have its status becoming valid: %v", deploymentName, err)
  661. }
  662. // Wait for the deployment to have a progress complete condition.
  663. if err = tester.waitForDeploymentWithCondition(deploymentutil.NewRSAvailableReason, apps.DeploymentProgressing); err != nil {
  664. t.Fatal(err)
  665. }
  666. }
  667. func TestOverlappingDeployments(t *testing.T) {
  668. s, closeFn, rm, dc, informers, c := dcSetup(t)
  669. defer closeFn()
  670. name := "test-overlapping-deployments"
  671. ns := framework.CreateTestingNamespace(name, s, t)
  672. defer framework.DeleteTestingNamespace(ns, s, t)
  673. replicas := int32(1)
  674. firstDeploymentName := "first-deployment"
  675. secondDeploymentName := "second-deployment"
  676. testers := []*deploymentTester{
  677. {t: t, c: c, deployment: newDeployment(firstDeploymentName, ns.Name, replicas)},
  678. {t: t, c: c, deployment: newDeployment(secondDeploymentName, ns.Name, replicas)},
  679. }
  680. // Start informer and controllers
  681. stopCh := make(chan struct{})
  682. defer close(stopCh)
  683. informers.Start(stopCh)
  684. go rm.Run(5, stopCh)
  685. go dc.Run(5, stopCh)
  686. // Create 2 deployments with overlapping selectors
  687. var err error
  688. var rss []*apps.ReplicaSet
  689. for _, tester := range testers {
  690. tester.deployment, err = c.AppsV1().Deployments(ns.Name).Create(tester.deployment)
  691. dname := tester.deployment.Name
  692. if err != nil {
  693. t.Fatalf("failed to create deployment %q: %v", dname, err)
  694. }
  695. // Wait for the deployment to be updated to revision 1
  696. if err = tester.waitForDeploymentRevisionAndImage("1", fakeImage); err != nil {
  697. t.Fatalf("failed to update deployment %q to revision 1: %v", dname, err)
  698. }
  699. // Make sure the deployment completes while manually marking its pods as ready at the same time
  700. if err = tester.waitForDeploymentCompleteAndMarkPodsReady(); err != nil {
  701. t.Fatalf("deployment %q failed to complete: %v", dname, err)
  702. }
  703. // Get replicaset of the deployment
  704. newRS, err := tester.getNewReplicaSet()
  705. if err != nil {
  706. t.Fatalf("failed to get new replicaset of deployment %q: %v", dname, err)
  707. }
  708. if newRS == nil {
  709. t.Fatalf("unable to find new replicaset of deployment %q", dname)
  710. }
  711. // Store the replicaset for future usage
  712. rss = append(rss, newRS)
  713. }
  714. // Both deployments should proceed independently, so their respective replicaset should not be the same replicaset
  715. if rss[0].UID == rss[1].UID {
  716. t.Fatalf("overlapping deployments should not share the same replicaset")
  717. }
  718. // Scale only the first deployment by 1
  719. newReplicas := replicas + 1
  720. testers[0].deployment, err = testers[0].updateDeployment(func(update *apps.Deployment) {
  721. update.Spec.Replicas = &newReplicas
  722. })
  723. if err != nil {
  724. t.Fatalf("failed updating deployment %q: %v", firstDeploymentName, err)
  725. }
  726. // Make sure the deployment completes after scaling
  727. if err := testers[0].waitForDeploymentCompleteAndMarkPodsReady(); err != nil {
  728. t.Fatalf("deployment %q failed to complete after scaling: %v", firstDeploymentName, err)
  729. }
  730. // Verify replicaset of both deployments has updated number of replicas
  731. for i, tester := range testers {
  732. rs, err := c.AppsV1().ReplicaSets(ns.Name).Get(rss[i].Name, metav1.GetOptions{})
  733. if err != nil {
  734. t.Fatalf("failed to get replicaset %q: %v", rss[i].Name, err)
  735. }
  736. if *rs.Spec.Replicas != *tester.deployment.Spec.Replicas {
  737. t.Errorf("expected replicaset %q of deployment %q has %d replicas, but found %d replicas", rs.Name, firstDeploymentName, *tester.deployment.Spec.Replicas, *rs.Spec.Replicas)
  738. }
  739. }
  740. }
  741. // Deployment should not block rollout when updating spec replica number and template at the same time.
  742. func TestScaledRolloutDeployment(t *testing.T) {
  743. s, closeFn, rm, dc, informers, c := dcSetup(t)
  744. defer closeFn()
  745. name := "test-scaled-rollout-deployment"
  746. ns := framework.CreateTestingNamespace(name, s, t)
  747. defer framework.DeleteTestingNamespace(ns, s, t)
  748. stopCh := make(chan struct{})
  749. defer close(stopCh)
  750. informers.Start(stopCh)
  751. go rm.Run(5, stopCh)
  752. go dc.Run(5, stopCh)
  753. // Create a deployment with rolling update strategy, max surge = 3, and max unavailable = 2
  754. var err error
  755. replicas := int32(10)
  756. tester := &deploymentTester{t: t, c: c, deployment: newDeployment(name, ns.Name, replicas)}
  757. tester.deployment.Spec.Strategy.RollingUpdate.MaxSurge = intOrStrP(3)
  758. tester.deployment.Spec.Strategy.RollingUpdate.MaxUnavailable = intOrStrP(2)
  759. tester.deployment, err = c.AppsV1().Deployments(ns.Name).Create(tester.deployment)
  760. if err != nil {
  761. t.Fatalf("failed to create deployment %q: %v", name, err)
  762. }
  763. if err = tester.waitForDeploymentRevisionAndImage("1", fakeImage); err != nil {
  764. t.Fatal(err)
  765. }
  766. if err = tester.waitForDeploymentCompleteAndMarkPodsReady(); err != nil {
  767. t.Fatalf("deployment %q failed to complete: %v", name, err)
  768. }
  769. // Record current replicaset before starting new rollout
  770. firstRS, err := tester.expectNewReplicaSet()
  771. if err != nil {
  772. t.Fatal(err)
  773. }
  774. // Update the deployment with another new image but do not mark the pods as ready to block new replicaset
  775. fakeImage2 := "fakeimage2"
  776. tester.deployment, err = tester.updateDeployment(func(update *apps.Deployment) {
  777. update.Spec.Template.Spec.Containers[0].Image = fakeImage2
  778. })
  779. if err != nil {
  780. t.Fatalf("failed updating deployment %q: %v", name, err)
  781. }
  782. if err = tester.waitForDeploymentRevisionAndImage("2", fakeImage2); err != nil {
  783. t.Fatal(err)
  784. }
  785. // Verify the deployment has minimum available replicas after 2nd rollout
  786. tester.deployment, err = c.AppsV1().Deployments(ns.Name).Get(name, metav1.GetOptions{})
  787. if err != nil {
  788. t.Fatalf("failed to get deployment %q: %v", name, err)
  789. }
  790. minAvailableReplicas := deploymentutil.MinAvailable(tester.deployment)
  791. if tester.deployment.Status.AvailableReplicas < minAvailableReplicas {
  792. t.Fatalf("deployment %q does not have minimum number of available replicas after 2nd rollout", name)
  793. }
  794. // Wait for old replicaset of 1st rollout to have desired replicas
  795. firstRS, err = c.AppsV1().ReplicaSets(ns.Name).Get(firstRS.Name, metav1.GetOptions{})
  796. if err != nil {
  797. t.Fatalf("failed to get replicaset %q: %v", firstRS.Name, err)
  798. }
  799. if err = tester.waitRSStable(firstRS); err != nil {
  800. t.Fatal(err)
  801. }
  802. // Wait for new replicaset of 2nd rollout to have desired replicas
  803. secondRS, err := tester.expectNewReplicaSet()
  804. if err != nil {
  805. t.Fatal(err)
  806. }
  807. if err = tester.waitRSStable(secondRS); err != nil {
  808. t.Fatal(err)
  809. }
  810. // Scale up the deployment and update its image to another new image simultaneously (this time marks all pods as ready)
  811. newReplicas := int32(20)
  812. fakeImage3 := "fakeimage3"
  813. tester.deployment, err = tester.updateDeployment(func(update *apps.Deployment) {
  814. update.Spec.Replicas = &newReplicas
  815. update.Spec.Template.Spec.Containers[0].Image = fakeImage3
  816. })
  817. if err != nil {
  818. t.Fatalf("failed updating deployment %q: %v", name, err)
  819. }
  820. if err = tester.waitForDeploymentRevisionAndImage("3", fakeImage3); err != nil {
  821. t.Fatal(err)
  822. }
  823. if err = tester.waitForDeploymentCompleteAndMarkPodsReady(); err != nil {
  824. t.Fatalf("deployment %q failed to complete: %v", name, err)
  825. }
  826. // Verify every replicaset has correct desiredReplicas annotation after 3rd rollout
  827. thirdRS, err := deploymentutil.GetNewReplicaSet(tester.deployment, c.AppsV1())
  828. if err != nil {
  829. t.Fatalf("failed getting new revision 3 replicaset for deployment %q: %v", name, err)
  830. }
  831. rss := []*apps.ReplicaSet{firstRS, secondRS, thirdRS}
  832. for _, curRS := range rss {
  833. curRS, err = c.AppsV1().ReplicaSets(ns.Name).Get(curRS.Name, metav1.GetOptions{})
  834. if err != nil {
  835. t.Fatalf("failed to get replicaset when checking desired replicas annotation: %v", err)
  836. }
  837. desired, ok := deploymentutil.GetDesiredReplicasAnnotation(curRS)
  838. if !ok {
  839. t.Fatalf("failed to retrieve desiredReplicas annotation for replicaset %q", curRS.Name)
  840. }
  841. if desired != *(tester.deployment.Spec.Replicas) {
  842. t.Fatalf("unexpected desiredReplicas annotation for replicaset %q: expected %d, got %d", curRS.Name, *(tester.deployment.Spec.Replicas), desired)
  843. }
  844. }
  845. // Update the deployment with another new image but do not mark the pods as ready to block new replicaset
  846. fakeImage4 := "fakeimage4"
  847. tester.deployment, err = tester.updateDeployment(func(update *apps.Deployment) {
  848. update.Spec.Template.Spec.Containers[0].Image = fakeImage4
  849. })
  850. if err != nil {
  851. t.Fatalf("failed updating deployment %q: %v", name, err)
  852. }
  853. if err = tester.waitForDeploymentRevisionAndImage("4", fakeImage4); err != nil {
  854. t.Fatal(err)
  855. }
  856. // Verify the deployment has minimum available replicas after 4th rollout
  857. tester.deployment, err = c.AppsV1().Deployments(ns.Name).Get(name, metav1.GetOptions{})
  858. if err != nil {
  859. t.Fatalf("failed to get deployment %q: %v", name, err)
  860. }
  861. minAvailableReplicas = deploymentutil.MinAvailable(tester.deployment)
  862. if tester.deployment.Status.AvailableReplicas < minAvailableReplicas {
  863. t.Fatalf("deployment %q does not have minimum number of available replicas after 4th rollout", name)
  864. }
  865. // Wait for old replicaset of 3rd rollout to have desired replicas
  866. thirdRS, err = c.AppsV1().ReplicaSets(ns.Name).Get(thirdRS.Name, metav1.GetOptions{})
  867. if err != nil {
  868. t.Fatalf("failed to get replicaset %q: %v", thirdRS.Name, err)
  869. }
  870. if err = tester.waitRSStable(thirdRS); err != nil {
  871. t.Fatal(err)
  872. }
  873. // Wait for new replicaset of 4th rollout to have desired replicas
  874. fourthRS, err := tester.expectNewReplicaSet()
  875. if err != nil {
  876. t.Fatal(err)
  877. }
  878. if err = tester.waitRSStable(fourthRS); err != nil {
  879. t.Fatal(err)
  880. }
  881. // Scale down the deployment and update its image to another new image simultaneously (this time marks all pods as ready)
  882. newReplicas = int32(5)
  883. fakeImage5 := "fakeimage5"
  884. tester.deployment, err = tester.updateDeployment(func(update *apps.Deployment) {
  885. update.Spec.Replicas = &newReplicas
  886. update.Spec.Template.Spec.Containers[0].Image = fakeImage5
  887. })
  888. if err != nil {
  889. t.Fatalf("failed updating deployment %q: %v", name, err)
  890. }
  891. if err = tester.waitForDeploymentRevisionAndImage("5", fakeImage5); err != nil {
  892. t.Fatal(err)
  893. }
  894. if err = tester.waitForDeploymentCompleteAndMarkPodsReady(); err != nil {
  895. t.Fatalf("deployment %q failed to complete: %v", name, err)
  896. }
  897. // Verify every replicaset has correct desiredReplicas annotation after 5th rollout
  898. fifthRS, err := deploymentutil.GetNewReplicaSet(tester.deployment, c.AppsV1())
  899. if err != nil {
  900. t.Fatalf("failed getting new revision 5 replicaset for deployment %q: %v", name, err)
  901. }
  902. rss = []*apps.ReplicaSet{thirdRS, fourthRS, fifthRS}
  903. for _, curRS := range rss {
  904. curRS, err = c.AppsV1().ReplicaSets(ns.Name).Get(curRS.Name, metav1.GetOptions{})
  905. if err != nil {
  906. t.Fatalf("failed to get replicaset when checking desired replicas annotation: %v", err)
  907. }
  908. desired, ok := deploymentutil.GetDesiredReplicasAnnotation(curRS)
  909. if !ok {
  910. t.Fatalf("failed to retrieve desiredReplicas annotation for replicaset %q", curRS.Name)
  911. }
  912. if desired != *(tester.deployment.Spec.Replicas) {
  913. t.Fatalf("unexpected desiredReplicas annotation for replicaset %q: expected %d, got %d", curRS.Name, *(tester.deployment.Spec.Replicas), desired)
  914. }
  915. }
  916. }
  917. func TestSpecReplicasChange(t *testing.T) {
  918. s, closeFn, rm, dc, informers, c := dcSetup(t)
  919. defer closeFn()
  920. name := "test-spec-replicas-change"
  921. ns := framework.CreateTestingNamespace(name, s, t)
  922. defer framework.DeleteTestingNamespace(ns, s, t)
  923. deploymentName := "deployment"
  924. replicas := int32(1)
  925. tester := &deploymentTester{t: t, c: c, deployment: newDeployment(deploymentName, ns.Name, replicas)}
  926. tester.deployment.Spec.Strategy.Type = apps.RecreateDeploymentStrategyType
  927. tester.deployment.Spec.Strategy.RollingUpdate = nil
  928. var err error
  929. tester.deployment, err = c.AppsV1().Deployments(ns.Name).Create(tester.deployment)
  930. if err != nil {
  931. t.Fatalf("failed to create deployment %q: %v", deploymentName, err)
  932. }
  933. // Start informer and controllers
  934. stopCh := make(chan struct{})
  935. defer close(stopCh)
  936. informers.Start(stopCh)
  937. go rm.Run(5, stopCh)
  938. go dc.Run(5, stopCh)
  939. // Scale up/down deployment and verify its replicaset has matching .spec.replicas
  940. if err = tester.scaleDeployment(2); err != nil {
  941. t.Fatal(err)
  942. }
  943. if err = tester.scaleDeployment(0); err != nil {
  944. t.Fatal(err)
  945. }
  946. if err = tester.scaleDeployment(1); err != nil {
  947. t.Fatal(err)
  948. }
  949. // Add a template annotation change to test deployment's status does update
  950. // without .spec.replicas change
  951. var oldGeneration int64
  952. tester.deployment, err = tester.updateDeployment(func(update *apps.Deployment) {
  953. oldGeneration = update.Generation
  954. update.Spec.RevisionHistoryLimit = pointer.Int32Ptr(4)
  955. })
  956. if err != nil {
  957. t.Fatalf("failed updating deployment %q: %v", tester.deployment.Name, err)
  958. }
  959. savedGeneration := tester.deployment.Generation
  960. if savedGeneration == oldGeneration {
  961. t.Fatalf("Failed to verify .Generation has incremented for deployment %q", deploymentName)
  962. }
  963. if err = tester.waitForObservedDeployment(savedGeneration); err != nil {
  964. t.Fatal(err)
  965. }
  966. }
  967. func TestDeploymentAvailableCondition(t *testing.T) {
  968. s, closeFn, rm, dc, informers, c := dcSetup(t)
  969. defer closeFn()
  970. name := "test-deployment-available-condition"
  971. ns := framework.CreateTestingNamespace(name, s, t)
  972. defer framework.DeleteTestingNamespace(ns, s, t)
  973. deploymentName := "deployment"
  974. replicas := int32(10)
  975. tester := &deploymentTester{t: t, c: c, deployment: newDeployment(deploymentName, ns.Name, replicas)}
  976. // Assign a high value to the deployment's minReadySeconds
  977. tester.deployment.Spec.MinReadySeconds = 3600
  978. // progressDeadlineSeconds must be greater than minReadySeconds
  979. tester.deployment.Spec.ProgressDeadlineSeconds = pointer.Int32Ptr(7200)
  980. var err error
  981. tester.deployment, err = c.AppsV1().Deployments(ns.Name).Create(tester.deployment)
  982. if err != nil {
  983. t.Fatalf("failed to create deployment %q: %v", deploymentName, err)
  984. }
  985. // Start informer and controllers
  986. stopCh := make(chan struct{})
  987. defer close(stopCh)
  988. informers.Start(stopCh)
  989. go rm.Run(5, stopCh)
  990. go dc.Run(5, stopCh)
  991. // Wait for the deployment to be observed by the controller and has at least specified number of updated replicas
  992. if err = tester.waitForDeploymentUpdatedReplicasGTE(replicas); err != nil {
  993. t.Fatal(err)
  994. }
  995. // Wait for the deployment to have MinimumReplicasUnavailable reason because the pods are not marked as ready
  996. if err = tester.waitForDeploymentWithCondition(deploymentutil.MinimumReplicasUnavailable, apps.DeploymentAvailable); err != nil {
  997. t.Fatal(err)
  998. }
  999. // Verify all replicas fields of DeploymentStatus have desired counts
  1000. if err = tester.checkDeploymentStatusReplicasFields(10, 10, 0, 0, 10); err != nil {
  1001. t.Fatal(err)
  1002. }
  1003. // Mark the pods as ready without waiting for the deployment to complete
  1004. if err = tester.markUpdatedPodsReadyWithoutComplete(); err != nil {
  1005. t.Fatal(err)
  1006. }
  1007. // Wait for number of ready replicas to equal number of replicas.
  1008. if err = tester.waitForReadyReplicas(); err != nil {
  1009. t.Fatal(err)
  1010. }
  1011. // Wait for the deployment to still have MinimumReplicasUnavailable reason within minReadySeconds period
  1012. if err = tester.waitForDeploymentWithCondition(deploymentutil.MinimumReplicasUnavailable, apps.DeploymentAvailable); err != nil {
  1013. t.Fatal(err)
  1014. }
  1015. // Verify all replicas fields of DeploymentStatus have desired counts
  1016. if err = tester.checkDeploymentStatusReplicasFields(10, 10, 10, 0, 10); err != nil {
  1017. t.Fatal(err)
  1018. }
  1019. // Update the deployment's minReadySeconds to a small value
  1020. tester.deployment, err = tester.updateDeployment(func(update *apps.Deployment) {
  1021. update.Spec.MinReadySeconds = 1
  1022. })
  1023. if err != nil {
  1024. t.Fatalf("failed updating deployment %q: %v", deploymentName, err)
  1025. }
  1026. // Wait for the deployment to notice minReadySeconds has changed
  1027. if err := tester.waitForObservedDeployment(tester.deployment.Generation); err != nil {
  1028. t.Fatal(err)
  1029. }
  1030. // Wait for the deployment to have MinimumReplicasAvailable reason after minReadySeconds period
  1031. if err = tester.waitForDeploymentWithCondition(deploymentutil.MinimumReplicasAvailable, apps.DeploymentAvailable); err != nil {
  1032. t.Fatal(err)
  1033. }
  1034. // Verify all replicas fields of DeploymentStatus have desired counts
  1035. if err = tester.checkDeploymentStatusReplicasFields(10, 10, 10, 10, 0); err != nil {
  1036. t.Fatal(err)
  1037. }
  1038. }
  1039. // Wait for deployment to automatically patch incorrect ControllerRef of RS
  1040. func testRSControllerRefPatch(t *testing.T, tester *deploymentTester, rs *apps.ReplicaSet, ownerReference *metav1.OwnerReference, expectedOwnerReferenceNum int) {
  1041. ns := rs.Namespace
  1042. rsClient := tester.c.AppsV1().ReplicaSets(ns)
  1043. rs, err := tester.updateReplicaSet(rs.Name, func(update *apps.ReplicaSet) {
  1044. update.OwnerReferences = []metav1.OwnerReference{*ownerReference}
  1045. })
  1046. if err != nil {
  1047. t.Fatalf("failed to update replicaset %q: %v", rs.Name, err)
  1048. }
  1049. if err := wait.PollImmediate(pollInterval, pollTimeout, func() (bool, error) {
  1050. newRS, err := rsClient.Get(rs.Name, metav1.GetOptions{})
  1051. if err != nil {
  1052. return false, err
  1053. }
  1054. return metav1.GetControllerOf(newRS) != nil, nil
  1055. }); err != nil {
  1056. t.Fatalf("failed to wait for controllerRef of the replicaset %q to become nil: %v", rs.Name, err)
  1057. }
  1058. newRS, err := rsClient.Get(rs.Name, metav1.GetOptions{})
  1059. if err != nil {
  1060. t.Fatalf("failed to obtain replicaset %q: %v", rs.Name, err)
  1061. }
  1062. controllerRef := metav1.GetControllerOf(newRS)
  1063. if controllerRef.UID != tester.deployment.UID {
  1064. t.Fatalf("controllerRef of replicaset %q has a different UID: Expected %v, got %v", newRS.Name, tester.deployment.UID, controllerRef.UID)
  1065. }
  1066. ownerReferenceNum := len(newRS.GetOwnerReferences())
  1067. if ownerReferenceNum != expectedOwnerReferenceNum {
  1068. t.Fatalf("unexpected number of owner references for replicaset %q: Expected %d, got %d", newRS.Name, expectedOwnerReferenceNum, ownerReferenceNum)
  1069. }
  1070. }
  1071. func TestGeneralReplicaSetAdoption(t *testing.T) {
  1072. s, closeFn, rm, dc, informers, c := dcSetup(t)
  1073. defer closeFn()
  1074. name := "test-general-replicaset-adoption"
  1075. ns := framework.CreateTestingNamespace(name, s, t)
  1076. defer framework.DeleteTestingNamespace(ns, s, t)
  1077. deploymentName := "deployment"
  1078. replicas := int32(1)
  1079. tester := &deploymentTester{t: t, c: c, deployment: newDeployment(deploymentName, ns.Name, replicas)}
  1080. var err error
  1081. tester.deployment, err = c.AppsV1().Deployments(ns.Name).Create(tester.deployment)
  1082. if err != nil {
  1083. t.Fatalf("failed to create deployment %q: %v", deploymentName, err)
  1084. }
  1085. // Start informer and controllers
  1086. stopCh := make(chan struct{})
  1087. defer close(stopCh)
  1088. informers.Start(stopCh)
  1089. go rm.Run(5, stopCh)
  1090. go dc.Run(5, stopCh)
  1091. // Wait for the Deployment to be updated to revision 1
  1092. if err := tester.waitForDeploymentRevisionAndImage("1", fakeImage); err != nil {
  1093. t.Fatal(err)
  1094. }
  1095. // Ensure the deployment completes while marking its pods as ready simultaneously
  1096. if err := tester.waitForDeploymentCompleteAndMarkPodsReady(); err != nil {
  1097. t.Fatal(err)
  1098. }
  1099. // Get replicaset of the deployment
  1100. rs, err := deploymentutil.GetNewReplicaSet(tester.deployment, c.AppsV1())
  1101. if err != nil {
  1102. t.Fatalf("failed to get replicaset of deployment %q: %v", deploymentName, err)
  1103. }
  1104. if rs == nil {
  1105. t.Fatalf("unable to find replicaset of deployment %q", deploymentName)
  1106. }
  1107. // When the only OwnerReference of the RS points to another type of API object such as statefulset
  1108. // with Controller=false, the deployment should add a second OwnerReference (ControllerRef) pointing to itself
  1109. // with Controller=true
  1110. var falseVar = false
  1111. ownerReference := metav1.OwnerReference{UID: uuid.NewUUID(), APIVersion: "apps/v1", Kind: "StatefulSet", Name: deploymentName, Controller: &falseVar}
  1112. testRSControllerRefPatch(t, tester, rs, &ownerReference, 2)
  1113. // When the only OwnerReference of the RS points to the deployment with Controller=false,
  1114. // the deployment should set Controller=true for the only OwnerReference
  1115. ownerReference = metav1.OwnerReference{UID: tester.deployment.UID, APIVersion: "apps/v1", Kind: "Deployment", Name: deploymentName, Controller: &falseVar}
  1116. testRSControllerRefPatch(t, tester, rs, &ownerReference, 1)
  1117. }
  1118. func testScalingUsingScaleSubresource(t *testing.T, tester *deploymentTester, replicas int32) {
  1119. ns := tester.deployment.Namespace
  1120. deploymentName := tester.deployment.Name
  1121. deploymentClient := tester.c.AppsV1().Deployments(ns)
  1122. deployment, err := deploymentClient.Get(deploymentName, metav1.GetOptions{})
  1123. if err != nil {
  1124. t.Fatalf("Failed to obtain deployment %q: %v", deploymentName, err)
  1125. }
  1126. scale, err := tester.c.AppsV1().Deployments(ns).GetScale(deploymentName, metav1.GetOptions{})
  1127. if err != nil {
  1128. t.Fatalf("Failed to obtain scale subresource for deployment %q: %v", deploymentName, err)
  1129. }
  1130. if scale.Spec.Replicas != *deployment.Spec.Replicas {
  1131. t.Fatalf("Scale subresource for deployment %q does not match .Spec.Replicas: expected %d, got %d", deploymentName, *deployment.Spec.Replicas, scale.Spec.Replicas)
  1132. }
  1133. if err := retry.RetryOnConflict(retry.DefaultBackoff, func() error {
  1134. scale, err := tester.c.AppsV1().Deployments(ns).GetScale(deploymentName, metav1.GetOptions{})
  1135. if err != nil {
  1136. return err
  1137. }
  1138. scale.Spec.Replicas = replicas
  1139. _, err = tester.c.AppsV1().Deployments(ns).UpdateScale(deploymentName, scale)
  1140. return err
  1141. }); err != nil {
  1142. t.Fatalf("Failed to set .Spec.Replicas of scale subresource for deployment %q: %v", deploymentName, err)
  1143. }
  1144. deployment, err = deploymentClient.Get(deploymentName, metav1.GetOptions{})
  1145. if err != nil {
  1146. t.Fatalf("Failed to obtain deployment %q: %v", deploymentName, err)
  1147. }
  1148. if *deployment.Spec.Replicas != replicas {
  1149. t.Fatalf(".Spec.Replicas of deployment %q does not match its scale subresource: expected %d, got %d", deploymentName, replicas, *deployment.Spec.Replicas)
  1150. }
  1151. }
  1152. func TestDeploymentScaleSubresource(t *testing.T) {
  1153. s, closeFn, rm, dc, informers, c := dcSetup(t)
  1154. defer closeFn()
  1155. name := "test-deployment-scale-subresource"
  1156. ns := framework.CreateTestingNamespace(name, s, t)
  1157. defer framework.DeleteTestingNamespace(ns, s, t)
  1158. deploymentName := "deployment"
  1159. replicas := int32(2)
  1160. tester := &deploymentTester{t: t, c: c, deployment: newDeployment(deploymentName, ns.Name, replicas)}
  1161. var err error
  1162. tester.deployment, err = c.AppsV1().Deployments(ns.Name).Create(tester.deployment)
  1163. if err != nil {
  1164. t.Fatalf("failed to create deployment %q: %v", deploymentName, err)
  1165. }
  1166. // Start informer and controllers
  1167. stopCh := make(chan struct{})
  1168. defer close(stopCh)
  1169. informers.Start(stopCh)
  1170. go rm.Run(5, stopCh)
  1171. go dc.Run(5, stopCh)
  1172. // Wait for the Deployment to be updated to revision 1
  1173. if err := tester.waitForDeploymentRevisionAndImage("1", fakeImage); err != nil {
  1174. t.Fatal(err)
  1175. }
  1176. // Ensure the deployment completes while marking its pods as ready simultaneously
  1177. if err := tester.waitForDeploymentCompleteAndMarkPodsReady(); err != nil {
  1178. t.Fatal(err)
  1179. }
  1180. // Use scale subresource to scale the deployment up to 3
  1181. testScalingUsingScaleSubresource(t, tester, 3)
  1182. // Use the scale subresource to scale the deployment down to 0
  1183. testScalingUsingScaleSubresource(t, tester, 0)
  1184. }
  1185. // This test verifies that the Deployment does orphan a ReplicaSet when the ReplicaSet's
  1186. // .Labels field is changed to no longer match the Deployment's selector. It also partially
  1187. // verifies that collision avoidance mechanism is triggered when a Deployment's new ReplicaSet
  1188. // is orphaned, even without PodTemplateSpec change. Refer comment below for more info:
  1189. // https://github.com/kubernetes/kubernetes/pull/59212#discussion_r166465113
  1190. func TestReplicaSetOrphaningAndAdoptionWhenLabelsChange(t *testing.T) {
  1191. s, closeFn, rm, dc, informers, c := dcSetup(t)
  1192. defer closeFn()
  1193. name := "test-replicaset-orphaning-and-adoption-when-labels-change"
  1194. ns := framework.CreateTestingNamespace(name, s, t)
  1195. defer framework.DeleteTestingNamespace(ns, s, t)
  1196. deploymentName := "deployment"
  1197. replicas := int32(1)
  1198. tester := &deploymentTester{t: t, c: c, deployment: newDeployment(deploymentName, ns.Name, replicas)}
  1199. var err error
  1200. tester.deployment, err = c.AppsV1().Deployments(ns.Name).Create(tester.deployment)
  1201. if err != nil {
  1202. t.Fatalf("failed to create deployment %q: %v", deploymentName, err)
  1203. }
  1204. // Start informer and controllers
  1205. stopCh := make(chan struct{})
  1206. defer close(stopCh)
  1207. informers.Start(stopCh)
  1208. go rm.Run(5, stopCh)
  1209. go dc.Run(5, stopCh)
  1210. // Wait for the Deployment to be updated to revision 1
  1211. if err := tester.waitForDeploymentRevisionAndImage("1", fakeImage); err != nil {
  1212. t.Fatal(err)
  1213. }
  1214. // Ensure the deployment completes while marking its pods as ready simultaneously
  1215. if err := tester.waitForDeploymentCompleteAndMarkPodsReady(); err != nil {
  1216. t.Fatal(err)
  1217. }
  1218. // Orphaning: deployment should remove OwnerReference from a RS when the RS's labels change to not match its labels
  1219. // Get replicaset of the deployment
  1220. rs, err := deploymentutil.GetNewReplicaSet(tester.deployment, c.AppsV1())
  1221. if err != nil {
  1222. t.Fatalf("failed to get replicaset of deployment %q: %v", deploymentName, err)
  1223. }
  1224. if rs == nil {
  1225. t.Fatalf("unable to find replicaset of deployment %q", deploymentName)
  1226. }
  1227. // Verify controllerRef of the replicaset is not nil and pointing to the deployment
  1228. controllerRef := metav1.GetControllerOf(rs)
  1229. if controllerRef == nil {
  1230. t.Fatalf("controllerRef of replicaset %q is nil", rs.Name)
  1231. }
  1232. if controllerRef.UID != tester.deployment.UID {
  1233. t.Fatalf("controllerRef of replicaset %q has a different UID: Expected %v, got %v", rs.Name, tester.deployment.UID, controllerRef.UID)
  1234. }
  1235. // Change the replicaset's labels to not match the deployment's labels
  1236. labelMap := map[string]string{"new-name": "new-test"}
  1237. rs, err = tester.updateReplicaSet(rs.Name, func(update *apps.ReplicaSet) {
  1238. update.Labels = labelMap
  1239. })
  1240. if err != nil {
  1241. t.Fatalf("failed to update replicaset %q: %v", rs.Name, err)
  1242. }
  1243. // Wait for the controllerRef of the replicaset to become nil
  1244. rsClient := tester.c.AppsV1().ReplicaSets(ns.Name)
  1245. if err = wait.PollImmediate(pollInterval, pollTimeout, func() (bool, error) {
  1246. rs, err = rsClient.Get(rs.Name, metav1.GetOptions{})
  1247. if err != nil {
  1248. return false, err
  1249. }
  1250. return metav1.GetControllerOf(rs) == nil, nil
  1251. }); err != nil {
  1252. t.Fatalf("failed to wait for controllerRef of replicaset %q to become nil: %v", rs.Name, err)
  1253. }
  1254. // Wait for the deployment to create a new replicaset
  1255. // This will trigger collision avoidance due to deterministic nature of replicaset name
  1256. // i.e., the new replicaset will have a name with different hash to preserve name uniqueness
  1257. var newRS *apps.ReplicaSet
  1258. if err = wait.PollImmediate(pollInterval, pollTimeout, func() (bool, error) {
  1259. newRS, err = deploymentutil.GetNewReplicaSet(tester.deployment, c.AppsV1())
  1260. if err != nil {
  1261. return false, fmt.Errorf("failed to get new replicaset of deployment %q after orphaning: %v", deploymentName, err)
  1262. }
  1263. return newRS != nil, nil
  1264. }); err != nil {
  1265. t.Fatalf("failed to wait for deployment %q to create a new replicaset after orphaning: %v", deploymentName, err)
  1266. }
  1267. if newRS.UID == rs.UID {
  1268. t.Fatalf("expect deployment %q to create a new replicaset different from the orphaned one, but it isn't", deploymentName)
  1269. }
  1270. // Adoption: deployment should add controllerRef to a RS when the RS's labels change to match its labels
  1271. // Change the old replicaset's labels to match the deployment's labels
  1272. rs, err = tester.updateReplicaSet(rs.Name, func(update *apps.ReplicaSet) {
  1273. update.Labels = testLabels()
  1274. })
  1275. if err != nil {
  1276. t.Fatalf("failed to update replicaset %q: %v", rs.Name, err)
  1277. }
  1278. // Wait for the deployment to adopt the old replicaset
  1279. if err = wait.PollImmediate(pollInterval, pollTimeout, func() (bool, error) {
  1280. rs, err := rsClient.Get(rs.Name, metav1.GetOptions{})
  1281. if err != nil {
  1282. return false, err
  1283. }
  1284. controllerRef = metav1.GetControllerOf(rs)
  1285. return controllerRef != nil && controllerRef.UID == tester.deployment.UID, nil
  1286. }); err != nil {
  1287. t.Fatalf("failed waiting for replicaset adoption by deployment %q to complete: %v", deploymentName, err)
  1288. }
  1289. }