knative-extensions / serving-progressive-rollout

Knative Serving extension to roll out the revision progressively
Apache License 2.0
3 stars 6 forks source link

If there is only one pod, or overratio set to 100, using resourceUtil strategy won't directly kill the old pods immediately #191

Open houshengbo opened 2 months ago

houshengbo commented 2 months ago

If there is only one pod, or overratio set to 100, using resourceUtil strategy won't directly kill the old pods immediately.

houshengbo commented 2 weeks ago

If the over-ratio is set to 100, the bug was fixed with the PR opened. However, if there is only one pod running for the old revision, there is nothing we can do from the logic, because it may take longer time for Kubernetes to add the field DeletionTimestamp into the pod.