Merge branch 'eb-remove-old-coverage-flags' into 'master'

Remove old coverage feature flags

See merge request gitlab-org/gitlab!43163

app/controllers/projects/ci/daily_build_group_report_results_controller.rb

@@ -6,7 +6,6 @@ class Projects::Ci::DailyBuildGroupReportResultsController < Projects::Applicati
   MAX_ITEMS = 1000
   REPORT_WINDOW = 90.days
 
-  before_action :validate_feature_flag!
   before_action :authorize_read_build_report_results!
   before_action :validate_param_type!
 
@@ -19,10 +18,6 @@ class Projects::Ci::DailyBuildGroupReportResultsController < Projects::Applicati
 
   private
 
-  def validate_feature_flag!
-    render_404 unless Feature.enabled?(:ci_download_daily_code_coverage, project, default_enabled: true)
-  end
-
   def validate_param_type!
     respond_422 unless allowed_param_types.include?(param_type)
   end

app/controllers/projects/graphs_controller.rb

@@ -57,7 +57,6 @@ class Projects::GraphsController < Projects::ApplicationController
   end
 
   def get_daily_coverage_options
-    return unless Feature.enabled?(:ci_download_daily_code_coverage, @project, default_enabled: true)
     return unless can?(current_user, :read_build_report_results, project)
 
     date_today = Date.current

app/services/ci/daily_build_group_report_result_service.rb

@@ -3,8 +3,6 @@
 module Ci
   class DailyBuildGroupReportResultService
     def execute(pipeline)
-      return unless Feature.enabled?(:ci_daily_code_coverage, pipeline.project, default_enabled: true)
-
       DailyBuildGroupReportResult.upsert_reports(coverage_reports(pipeline))
     end
 

config/feature_flags/development/ci_daily_code_coverage.yml

----
-name: ci_daily_code_coverage
-introduced_by_url: 
-rollout_issue_url: 
-group: 
-type: development
-default_enabled: true

config/feature_flags/development/ci_download_daily_code_coverage.yml

----
-name: ci_download_daily_code_coverage
-introduced_by_url: 
-rollout_issue_url: 
-group: 
-type: development
-default_enabled: true