Skip to content
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

Support for overriding primary scaler replicas #1343

Merged
merged 1 commit into from
Jan 17, 2023

Conversation

relu
Copy link
Member

@relu relu commented Jan 10, 2023

Adding support for overriding the primary scaler replica count via .spec.autoscalerRef.primaryScalerReplicas, a feature that would enable users to define a different scaling configuration for the primary.

This can be useful in the situation where the user does not want to scale the canary workload to the exact same size as the primary, especially when opting for a canary deployment pattern where only a small portion of traffic is routed to the canary workload pods.

@codecov-commenter
Copy link

Codecov Report

Base: 54.24% // Head: 54.34% // Increases project coverage by +0.10% 🎉

Coverage data is based on head (204ebea) compared to base (58270dd).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1343      +/-   ##
==========================================
+ Coverage   54.24%   54.34%   +0.10%     
==========================================
  Files          84       84              
  Lines       10016    10038      +22     
==========================================
+ Hits         5433     5455      +22     
  Misses       3927     3927              
  Partials      656      656              
Impacted Files Coverage Δ
pkg/canary/hpa_reconciler.go 79.13% <100.00%> (+1.35%) ⬆️
pkg/canary/scaled_object_reconciler.go 75.51% <100.00%> (+1.40%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@relu relu force-pushed the autoscaler-replicas-overrides branch from 204ebea to 9963839 Compare January 10, 2023 15:15
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's introduce here https://github.com/fluxcd/flagger/blob/main/docs/gitbook/usage/how-it-works.md#canary-target a section for the autoscaler and explain its new behaviour.

@stefanprodan stefanprodan added the kind/enhancement Improvement request for an existing feature label Jan 16, 2023
@relu relu force-pushed the autoscaler-replicas-overrides branch from 9963839 to 5e29dec Compare January 16, 2023 17:37
Adding support for overriding the primary scaler replica count via
.spec.autoscalerRef.primaryScalerReplicas, a feature which would enable
users to define a different scaling configurations for the primary.

This can be useful in the situation where the user does not want to
scale the canary workload to the exact same size as the primary,
especially when opting for a canary deployment pattern where only a
small portion of traffic is routed to the canary workload pods.

Signed-off-by: Aurel Canciu <[email protected]>
@relu relu force-pushed the autoscaler-replicas-overrides branch from 5e29dec to 5d3ab05 Compare January 16, 2023 17:47
Copy link
Member

@aryan9600 aryan9600 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, lgtm! 🚀

Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thanks @relu 🏅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Improvement request for an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants