-
Notifications
You must be signed in to change notification settings - Fork 744
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
Introduce ScalerReconciler
and refactor HPA reconciliation
#1211
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1211 +/- ##
==========================================
- Coverage 54.96% 54.77% -0.20%
==========================================
Files 79 80 +1
Lines 6642 6839 +197
==========================================
+ Hits 3651 3746 +95
- Misses 2418 2514 +96
- Partials 573 579 +6
Continue to review full report at Codecov.
|
…troller Signed-off-by: Sanskar Jaiswal <[email protected]>
Signed-off-by: Sanskar Jaiswal <[email protected]>
Signed-off-by: Sanskar Jaiswal <[email protected]>
5d052fc
to
9b97bff
Compare
Signed-off-by: Sanskar Jaiswal <[email protected]> for objectmeta as well
3fa1892
to
e0e2d5c
Compare
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.
LGTM
Thanks @aryan9600 🏅
This PR introduces a new interface
ScalerReconciler
, which provides methods to reconcile a scaler like HPAs, or KEDA ScaledObjects. For now, the existing HPA reconciliation logic has been refactored to implement this, is moved out of the deployment controller and is used directly in the main reconciliation loop.