fix: check for OK response + don't create empty labels selector objec…

…t when empty when upserting Advanced Config (#299)

* fix: check for OK response when upserting Advanced Config

* fix: don't create empty labels selector object when empty

* fixup! fix: check for OK response when upserting Advanced Config

castai/resource_eviction_config.go

@@ -214,9 +214,9 @@ func upsertEvictionConfigs(ctx context.Context, data *schema.ResourceData, meta
 		"application/json",
 		bytes.NewReader(evictorAdvancedConfigJson),
 	)
-	if err != nil || resp.JSON200 == nil {
+	if checkErr := sdk.CheckOKResponse(resp, err); checkErr != nil {
 		log.Printf("[ERROR] Failed to upsert evictor advanced config: %v", err)
-		return err
+		return checkErr
 	}
 	err = data.Set(FieldEvictorAdvancedConfig, flattenEvictionConfig(resp.JSON200.EvictionConfig))
 	if err != nil {
@@ -421,6 +421,10 @@ func toPodSelector(in interface{}) (*sdk.CastaiEvictorV1PodSelector, error) {
 				return nil, err
 			}
 
+			if mls == nil || len(mls.AdditionalProperties) == 0 {
+				continue
+			}
+
 			if out.LabelSelector == nil {
 				out.LabelSelector = &sdk.CastaiEvictorV1LabelSelector{}
 			}

castai/resource_rebalancing_job.go

@@ -189,7 +189,7 @@ func getRebalancingJobByScheduleName(ctx context.Context, client *sdk.ClientWith
 		return nil, fmt.Errorf("getting schedule: %w", err)
 	}
 
-	resp, err := client.ScheduledRebalancingAPIListRebalancingJobsWithResponse(ctx, clusterID)
+	resp, err := client.ScheduledRebalancingAPIListRebalancingJobsWithResponse(ctx, clusterID, &sdk.ScheduledRebalancingAPIListRebalancingJobsParams{})
 	if checkErr := sdk.CheckOKResponse(resp, err); checkErr != nil {
 		return nil, checkErr
 	}