Merge branch...

Merge branch '464300-follow-up-from-pep-mvc-define-ssot-for-pipeline-rejected-by-rules' into 'master' 

Refactor check for execution policy pipelines filtered by rules

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/159235



Merged-by: Fabio Pitino <fpitino@gitlab.com>
Approved-by: Fabio Pitino <fpitino@gitlab.com>
Co-authored-by: Martin Čavoj <mcavoj@gitlab.com>

ee/lib/ee/gitlab/ci/pipeline/chain/pipeline_execution_policies/find_configs.rb

@@ -40,13 +40,11 @@ def perform!
                 command.pipeline_execution_policies = []
                 pipeline_execution_policy_configs.each do |config|
                   response = create_pipeline(config.content)
+                  pipeline = response.payload
 
                   if response.success?
-                    command.pipeline_execution_policies << PipelineExecutionPolicy.new(
-                      response.payload,
-                      config.strategy
-                    )
-                  elsif pipeline_filtered_by_rules?(response.payload)
+                    command.pipeline_execution_policies << PipelineExecutionPolicy.new(pipeline, config.strategy)
+                  elsif pipeline.filtered_as_empty?
                     # no-op: we ignore empty pipelines
                   else
                     return error("Pipeline execution policy error: #{response.message}", failure_reason: :config_error)
@@ -77,11 +75,6 @@ def create_pipeline(content)
                     # Additional parameters will be added in https://gitlab.com/gitlab-org/gitlab/-/issues/462004
                   )
               end
-
-              def pipeline_filtered_by_rules?(pipeline)
-                pipeline.failure_reason.present? &&
-                  !::Enums::Ci::Pipeline.persistable_failure_reason?(pipeline.failure_reason.to_sym)
-              end
             end
           end
         end