Merge pull request kruize#1470 from shekhar316/update-model

[Kruize VPA Integration] Updating Default Model to Short Term Performance
dinogun · Jan 28, 2025 · 19bc624 · 19bc624
2 parents bf492b6 + 5c89965
commit 19bc624
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/main/java/com/autotune/analyzer/recommendations/updater/vpa/VpaUpdaterImpl.java b/src/main/java/com/autotune/analyzer/recommendations/updater/vpa/VpaUpdaterImpl.java
@@ -246,13 +246,13 @@ private List<RecommendedContainerResources>  convertRecommendationsToContainerPo
             } else {
                 for (MappedRecommendationForTimestamp value : recommendationData.values()) {
                     /*
-                     * Fetching Short Term Cost Recommendations By Default
+                     * The short-term performance recommendations is currently the default for VPA and is hardcoded.
                      * TODO:// Implement functionality to choose the desired term and model
                      **/
                     TermRecommendations termRecommendations = value.getShortTermRecommendations();
                     HashMap<AnalyzerConstants.ResourceSetting,
                             HashMap<AnalyzerConstants.RecommendationItem,
-                                    RecommendationConfigItem>> recommendationsConfig = termRecommendations.getCostRecommendations().getConfig();
+                                    RecommendationConfigItem>> recommendationsConfig = termRecommendations.getPerformanceRecommendations().getConfig();
 
                     Double cpuRecommendationValue = recommendationsConfig.get(AnalyzerConstants.ResourceSetting.requests).get(AnalyzerConstants.RecommendationItem.CPU).getAmount();
                     Double memoryRecommendationValue = recommendationsConfig.get(AnalyzerConstants.ResourceSetting.requests).get(AnalyzerConstants.RecommendationItem.MEMORY).getAmount();