Skip to content

Commit 963e4f2

Browse files
authored
fix: Revert "fix(controller): rollout stuck in Progressing. fixes #3988" (#4201)
Revert "fix(controller): rollout stuck in `Progressing`. fixes #3988 (#4072)" This reverts commit f4289b7.
1 parent 38c3ff3 commit 963e4f2

File tree

2 files changed

+0
-11
lines changed

2 files changed

+0
-11
lines changed

rollout/bluegreen_test.go

-1
Original file line numberDiff line numberDiff line change
@@ -1406,7 +1406,6 @@ func TestBlueGreenReadyToScaleDownOldReplica(t *testing.T) {
14061406
f.objects = append(f.objects, r2)
14071407
f.serviceLister = append(f.serviceLister, s)
14081408

1409-
f.expectPatchReplicaSetAction(rs2)
14101409
updatedRSIndex := f.expectUpdateReplicaSetAction(rs2)
14111410
patchIndex := f.expectPatchRolloutAction(r2)
14121411
f.run(getKey(r2, t))

rollout/replicaset.go

-10
Original file line numberDiff line numberDiff line change
@@ -107,16 +107,6 @@ func (c *rolloutContext) removeScaleDownDeadlines() error {
107107
toRemove = append(toRemove, c.stableRS)
108108
}
109109
}
110-
for _, rs := range c.otherRSs {
111-
remainScaleDownDeadlines, err := replicasetutil.GetTimeRemainingBeforeScaleDownDeadline(rs)
112-
if err != nil {
113-
c.log.Warnf("%v", err)
114-
continue
115-
}
116-
if replicasetutil.HasScaleDownDeadline(rs) && remainScaleDownDeadlines == nil {
117-
toRemove = append(toRemove, rs)
118-
}
119-
}
120110
for _, rs := range toRemove {
121111
err := c.removeScaleDownDelay(rs)
122112
if err != nil {

0 commit comments

Comments
 (0)