Revert "feat: support granular aggressive mode config (#458)"

This reverts commit 83288cf.

Author: vladklokun

castai/data_source_resource_rebalancing_schedule_test.go

@@ -50,10 +50,7 @@ func TestRebalancingScheduleDataSourceRead(t *testing.T) {
                     "evictGracefully": false,
                     "aggressiveMode": false,
                     "aggressiveModeConfig": {
-                        "ignoreLocalPersistentVolumes": true,
-                        "ignoreProblemJobPods": true,
-                        "ignoreProblemRemovalDisabledPods": true,
-                        "ignoreProblemPodsWithoutController": true
+                        "ignoreLocalPersistentVolumes": true	
                     }
                 },
                 "numTargetedNodes": 20,
@@ -134,9 +131,6 @@ launch_configuration.# = 1
 launch_configuration.0.aggressive_mode = false
 launch_configuration.0.aggressive_mode_config.# = 1
 launch_configuration.0.aggressive_mode_config.0.ignore_local_persistent_volumes = true
-launch_configuration.0.aggressive_mode_config.0.ignore_problem_job_pods = true
-launch_configuration.0.aggressive_mode_config.0.ignore_problem_pods_without_controller = true
-launch_configuration.0.aggressive_mode_config.0.ignore_problem_removal_disabled_pods = true
 launch_configuration.0.execution_conditions.# = 1
 launch_configuration.0.execution_conditions.0.achieved_savings_percentage = 15
 launch_configuration.0.execution_conditions.0.enabled = true

castai/resource_rebalancing_schedule.go

@@ -131,22 +131,7 @@ func resourceRebalancingSchedule() *schema.Resource {
 									"ignore_local_persistent_volumes": {
 										Type:        schema.TypeBool,
 										Required:    true,
-										Description: "Rebalance workloads that use local-path Persistent Volumes. THIS WILL RESULT IN DATA LOSS.",
-									},
-									"ignore_problem_job_pods": {
-										Type:        schema.TypeBool,
-										Required:    true,
-										Description: "Pods spawned by Jobs or CronJobs will not prevent the Rebalancer from deleting a node on which they run. WARNING: When true, pods spawned by Jobs or CronJobs will be terminated if the Rebalancer picks a node that runs them. As such, they are likely to lose their progress.",
-									},
-									"ignore_problem_removal_disabled_pods": {
-										Type:        schema.TypeBool,
-										Required:    true,
-										Description: `Pods that are marked with "removal disabled" will not prevent the Rebalancer from deleting a node on which they run. WARNING: When true, such pods will be evicted and disrupted.`,
-									},
-									"ignore_problem_pods_without_controller": {
-										Type:        schema.TypeBool,
-										Required:    true,
-										Description: "Pods that don't have a controller (bare pods) will not prevent the Rebalancer from deleting a node on which they run. WARNING: When true, such pods might not restart, since they have no controller to do it.",
+										Description: "Rebalance workloads using local-path Persistent Volumes. THIS WILL RESULT IN DATA LOSS.",
 									},
 								},
 							},
@@ -329,10 +314,7 @@ func stateToSchedule(d *schema.ResourceData) (*sdk.ScheduledrebalancingV1Rebalan
 		aggressiveModeConfigSection := launchConfigurationData["aggressive_mode_config"].([]any)
 		if len(aggressiveModeConfigSection) != 0 {
 			aggressiveModeConfig = &sdk.ScheduledrebalancingV1AggressiveModeConfig{
-				IgnoreLocalPersistentVolumes:       lo.ToPtr(aggressiveModeConfigSection[0].(map[string]any)["ignore_local_persistent_volumes"].(bool)),
-				IgnoreProblemJobPods:               lo.ToPtr(aggressiveModeConfigSection[0].(map[string]any)["ignore_problem_job_pods"].(bool)),
-				IgnoreProblemRemovalDisabledPods:   lo.ToPtr(aggressiveModeConfigSection[0].(map[string]any)["ignore_problem_removal_disabled_pods"].(bool)),
-				IgnoreProblemPodsWithoutController: lo.ToPtr(aggressiveModeConfigSection[0].(map[string]any)["ignore_problem_pods_without_controller"].(bool)),
+				IgnoreLocalPersistentVolumes: lo.ToPtr(aggressiveModeConfigSection[0].(map[string]any)["ignore_local_persistent_volumes"].(bool)),
 			}
 		}
 
@@ -381,10 +363,7 @@ func scheduleToState(schedule *sdk.ScheduledrebalancingV1RebalancingSchedule, d
 		if schedule.LaunchConfiguration.RebalancingOptions.AggressiveModeConfig != nil {
 			launchConfig["aggressive_mode_config"] = []map[string]any{
 				{
-					"ignore_local_persistent_volumes":        schedule.LaunchConfiguration.RebalancingOptions.AggressiveModeConfig.IgnoreLocalPersistentVolumes,
-					"ignore_problem_job_pods":                schedule.LaunchConfiguration.RebalancingOptions.AggressiveModeConfig.IgnoreProblemJobPods,
-					"ignore_problem_removal_disabled_pods":   schedule.LaunchConfiguration.RebalancingOptions.AggressiveModeConfig.IgnoreProblemRemovalDisabledPods,
-					"ignore_problem_pods_without_controller": schedule.LaunchConfiguration.RebalancingOptions.AggressiveModeConfig.IgnoreProblemPodsWithoutController,
+					"ignore_local_persistent_volumes": schedule.LaunchConfiguration.RebalancingOptions.AggressiveModeConfig.IgnoreLocalPersistentVolumes,
 				},
 			}
 		}

castai/resource_rebalancing_schedule_test.go

@@ -31,9 +31,6 @@ func TestAccResourceRebalancingSchedule_basic(t *testing.T) {
 					resource.TestCheckResourceAttr("castai_rebalancing_schedule.test", "schedule.0.cron", "1 4 * * *"),
 					resource.TestCheckResourceAttr("castai_rebalancing_schedule.test", "launch_configuration.0.aggressive_mode", "true"),
 					resource.TestCheckResourceAttr("castai_rebalancing_schedule.test", "launch_configuration.0.aggressive_mode_config.0.ignore_local_persistent_volumes", "true"),
-					resource.TestCheckResourceAttr("castai_rebalancing_schedule.test", "launch_configuration.0.aggressive_mode_config.0.ignore_problem_job_pods", "true"),
-					resource.TestCheckResourceAttr("castai_rebalancing_schedule.test", "launch_configuration.0.aggressive_mode_config.0.ignore_problem_removal_disabled_pods", "true"),
-					resource.TestCheckResourceAttr("castai_rebalancing_schedule.test", "launch_configuration.0.aggressive_mode_config.0.ignore_problem_pods_without_controller", "true"),
 				),
 			},
 			{
@@ -101,9 +98,6 @@ resource "castai_rebalancing_schedule" "test" {
 		aggressive_mode = true
 		aggressive_mode_config {
       		ignore_local_persistent_volumes = true
-      		ignore_problem_job_pods = true
-      		ignore_problem_removal_disabled_pods = true
-      		ignore_problem_pods_without_controller = true
     	}
 		selector = jsonencode({
 			nodeSelectorTerms = [{