-
Notifications
You must be signed in to change notification settings - Fork 4.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Preserve scaling factor of prior deployment #2097
Preserve scaling factor of prior deployment #2097
Conversation
When scaling up a new deployment and a prior deployment exists, preserve the replica count of the prior deployment. If no prior deployment exists, use the template replica count.
@ironcladlou overall LGTM |
// DesiredReplicasAnnotation represents the desired number of replicas for a | ||
// new deployment. | ||
// TODO: This should be made public upstream. | ||
DesiredReplicasAnnotation = "kubectl.kubernetes.io/desired-replicas" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there an issue or pull for that?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will do that now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[merge] |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/1825/) (Image: devenv-fedora_1451) |
[Test]ing while waiting on the merge queue |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/2037/) |
Evaluated for origin up to 64ee9f7 |
…tor-2 Merged by openshift-bot
When scaling up a new deployment and a prior deployment exists,
preserve the replica count of the prior deployment. If no prior
deployment exists, use the template replica count.