diff --git a/flyteadmin/pkg/manager/impl/validation/execution_validator.go b/flyteadmin/pkg/manager/impl/validation/execution_validator.go index f7c88b2376..b01e878b5b 100644 --- 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 index 7b569fae06..356231b73e 100644 --- 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 }