argo-rollouts icon indicating copy to clipboard operation
argo-rollouts copied to clipboard

`scaleDownDelaySeconds: 0` keeps old ReplicaSet running but should scale it down immediately

Open AndiDog opened this issue 2 years ago • 5 comments

Summary

With scaleDownDelaySeconds: 0 on a blue-green rollout, and clicking Promote-Full in Argo CD UI for faster deployment, the operator kept the old ReplicaSet running without scaling it down or terminating it (and its pods). We would expect the number of seconds to be respected, so zero means "immediately".

In contrast, any non-zero number like scaleDownDelaySeconds: 1 worked fine in our pre-production testing.

There are other reported issues where the meaning of zero was understood differently, so it's important to document it correctly and fix the behavior accordingly. See https://github.com/argoproj/argo-rollouts/issues/1639 and https://github.com/argoproj/argo-rollouts/issues/1841, for example.

Diagnostics

argo-rollouts v1.1.1

(I cannot offer logs right now)


Message from the maintainers:

Impacted by this bug? Give it a 👍. We prioritize the issues with the most 👍.

AndiDog avatar Apr 22 '22 21:04 AndiDog

So there is some documentation around this here https://argoproj.github.io/argo-rollouts/features/scaledown-aborted-rs/ but it seems to focus around when it is aborted which it might fall into that logic when clicking full promotion. I will try to take a look at this at some point to see if the behavior is correct and also that it matches what is documented.

zachaller avatar Apr 22 '22 22:04 zachaller

I think scaleDownDelaySeconds: 0 means permanent retention, and I want to add manual deletion action.

kzcPo avatar May 16 '22 01:05 kzcPo

We have also experience this and were surprised by it. We assumed by setting it to 0, there would be no delay and the ReplicaSet would be terminated immediately. We also ended up changing it to 1 to get closer towards the desired behavior.

mitchellhuang avatar Jun 17 '22 17:06 mitchellhuang

Another option if you want the replica sets to scale down immediately is to use scaleDownDelaySeconds: -1.

This is what we moved to since 0 keeps them indefinitely.

coolbry95 avatar Aug 05 '22 20:08 coolbry95

你好,我已经收到邮件!

kzcPo avatar Aug 05 '22 20:08 kzcPo

This issue is stale because it has been open 60 days with no activity.

github-actions[bot] avatar Feb 20 '23 02:02 github-actions[bot]

你好,我已经收到邮件!

kzcPo avatar Feb 20 '23 02:02 kzcPo

This issue is stale because it has been open 60 days with no activity.

github-actions[bot] avatar Apr 23 '23 02:04 github-actions[bot]

你好,我已经收到邮件!

kzcPo avatar Apr 23 '23 02:04 kzcPo

This issue is stale because it has been open 60 days with no activity.

github-actions[bot] avatar Jun 23 '23 02:06 github-actions[bot]

你好,我已经收到邮件!

kzcPo avatar Jun 23 '23 02:06 kzcPo

This issue is stale because it has been open 60 days with no activity.

github-actions[bot] avatar Aug 24 '23 02:08 github-actions[bot]