Remove uses of validate in flyteadmin

Signed-off-by: Eduardo Apolinario <[email protected]>
flyteorg · Dec 27, 2023 · 7db165e · 7db165e
1 parent 8a8e47d
commit 7db165e
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 10 deletions.
diff --git a/flyteadmin/pkg/manager/impl/validation/execution_validator.go b/flyteadmin/pkg/manager/impl/validation/execution_validator.go
@@ -68,11 +68,6 @@ func ValidateExecutionRequest(ctx context.Context, request admin.ExecutionCreate
 			return err
 		}
 	}
-	// TODO: Remove redundant validation with the rest of the method.
-	// This final call to validating the request ensures the notification types are expected.
-	if err := request.Validate(); err != nil {
-		return err
-	}
 	return nil
 }
 

diff --git a/flyteadmin/pkg/manager/impl/validation/launch_plan_validator.go b/flyteadmin/pkg/manager/impl/validation/launch_plan_validator.go
@@ -55,11 +55,6 @@ func ValidateLaunchPlan(ctx context.Context,
 			return err
 		}
 	}
-	// TODO: Remove redundant validation that occurs with launch plan and the validate method for the message.
-	// Ensure the notification types are validated.
-	if err := request.Validate(); err != nil {
-		return err
-	}
 	return nil
 }