github/flagger
1
0
Fork 0
mirror of https://github.com/fluxcd/flagger.git synced 2026-04-15 06:57:34 +00:00
Code Issues Packages Projects Releases Wiki Activity

If HPA isn't set and replicas are not specified, it uses the primary replicas when scaling up the canary

Browse Source 
Signed-off-by: Moscagus <gustavo.varisco@gmail.com>
This commit is contained in:
Moscagus
2022-03-09 22:16:47 -03:00
parent cf8fe94fca
commit f7fd6cce8c
1 changed files with 11 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkg/canary/deployment_controller.go
+11
View File
@@ -208,6 +208,17 @@ func (c *DeploymentController) ScaleFromZero(cd *flaggerv1.Canary) error {
replicas := int32p(1)
if dep.Spec.Replicas != nil && *dep.Spec.Replicas > 0 {
replicas = dep.Spec.Replicas
} else if cd.Spec.AutoscalerRef == nil {
// If HPA isn't set and replicas are not specified, it uses the primary replicas when scaling up the canary
primaryName := fmt.Sprintf("%s-primary", targetName)
primary, err := c.kubeClient.AppsV1().Deployments(cd.Namespace).Get(context.TODO(), primaryName, metav1.GetOptions{})
if err != nil {
return fmt.Errorf("deployment %s.%s get query error: %w", primaryName, cd.Namespace, err)
}
if primary.Spec.Replicas != nil && *primary.Spec.Replicas > 0 {
replicas = primary.Spec.Replicas
}
}
depCopy := dep.DeepCopy()
depCopy.Spec.Replicas = replicas