[go: up one dir, main page]

Remove ci_improved_destroy_old_pipelines_worker feature flag

Fabio Pitino - PTO until Jan 1requested to merge
remove-ci-improved-destroy-old-pipelines-worker-ff into master

What does this MR do and why?

This MR removes the ci_improved_destroy_old_pipelines_worker feature flag as part of the rollout completion process outlined in #571100 (closed).

The feature flag has been enabled in production and is stable. This change makes the improved logic the default behavior.

Changes

  • Removed the feature flag YAML definition from config/feature_flags/gitlab_com_derisk/ci_improved_destroy_old_pipelines_worker.yml
  • Updated Ci::DestroyOldPipelinesWorker to always use the improved logic:
    • Deletes pipelines directly without canceling them
    • Skips triggering callbacks for better performance
    • Requeues the project if there are more pipelines to remove
  • Removed the use_improved_logic? method that checked the feature flag
  • Updated specs to remove the feature flag test context

Related issues

Closes #571100 (closed)

Checklist

  • Feature flag YAML definition removed
  • All references to the feature flag removed from application code
  • Specs updated to remove feature flag contexts
  • Changelog entry included

Merge request reports