-
Notifications
You must be signed in to change notification settings - Fork 743
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
fix: reverse hpa ScalingLimited error condition #1366
base: main
Are you sure you want to change the base?
fix: reverse hpa ScalingLimited error condition #1366
Conversation
Signed-off-by: Antoine Deschênes <antoine.deschenes@linux.com>
@@ -46,7 +46,7 @@ var coreAnalyzerMap = map[string]common.IAnalyzer{ | |||
} | |||
|
|||
var additionalAnalyzerMap = map[string]common.IAnalyzer{ | |||
"HorizontalPodAutoScaler": HpaAnalyzer{}, | |||
"HorizontalPodAutoscaler": HpaAnalyzer{}, |
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.
This would break k8sgpt filters add
, maybe I should remove it?
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.
Are you saying because this is using v2 api it would break on default?
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.
I’m sorry, this wasn't clear. I mean that renaming the filter won’t align with the active_filters
value used in an existing k8sgpt.yaml
config file. This might need a config migration.
Should there also be a specific case for kubernetes/kubernetes#116387 |
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.
Need to figure out if this needs us to update the config
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1366 +/- ##
==========================================
- Coverage 34.76% 34.23% -0.54%
==========================================
Files 94 97 +3
Lines 6342 6511 +169
==========================================
+ Hits 2205 2229 +24
- Misses 4046 4189 +143
- Partials 91 93 +2 ☔ View full report in Codecov by Sentry. |
Closes #
📑 Description
k8sgpt currently outputs
- Error: the desired count is within the acceptable range
when an HPA is working properly. This is due to the "ScalingLimited" condition to be reversed:This also fixes a typo in the analyzer's name not matching the
kind
, although that could be removed for another time.✅ Checks
ℹ Additional Information