Skip to content

Unite Clickhouse CI data ingestion FF

Vladimir Shushlin requested to merge vshushlin/ch-ingestion-unite-ff into master

What does this MR do and why?

Unite Clickhouse CI data ingestion FF

We had 2 feature flags before:

  • generate_ci_finished_builds_sync_events: controls creation of Ci::FinishedBuildChSyncEvent
  • ci_data_ingestion_to_click_house: controls consumption of Ci::FinishedBuildChSyncEvent and syncing them the ClickHouse database

That made sense while we were rolling that out do GitLab.com, especially since we enabled the first one before we merged the data sync

Now when it's finished we want to test the same functionality with a few selected self-managed clients.

For them having 2 feature flags just complicates the process without much benefit

So this MR just unite the FFs, they are both enabled on staging/production: [Feature flag] Rollout of `generate_ci_finished... (#424866 - closed)

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports