Skip to content

FF wrapping SettingsBasedUpdateService

Gal Katzrequested to merge
gkatz_setting_changed_worker_fix_wrap_in_ff into master

What does this MR do and why?

Replace the existing post_pipeline_analyzer_status_updates feature flag with a new one wrapping SettingsBasedUpdateService. The old flag is already enabled globally and used elsewhere, so a dedicated flag is needed to safely re-enable ScheduleSettingChangedUpdateWorker following a recent performance incident.

Changelog: added
EE: true

References

Screenshots or screen recordings

Before After

How to set up and validate locally

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Relates to:
[FF] `group_settings_based_update_worker` -- wr... (#553607) • Gal Katz • 18.6
ScheduleSettingChangedUpdateWorker performance ... (#553362 - closed) • Gal Katz • 18.2
Safely re-enable Security::AnalyzersStatus::Set... (gitlab-com/gl-infra/production-engineering#27026 - closed) • Or Gal

Edited by Gal Katz

Merge request reports