Skip to content

Enable ClickHouse migrations by default

Vladimir Shushlin requested to merge vshushlin/enable-ch-migrations into master

What does this MR do and why?

Enable ClickHouse migrations by default

It turns out that we run migration differently on .com. See more in Clickhouse - automatically run migrations durin... (#428126 - moved)

So this feature flag can't really be tested on .com

On the self-managed we migrate CH only if it's configured:

https://gitlab.com/gitlab-org/gitlab/blob/716a459d47bab3945e6a86c9236680f424fefa68/lib/tasks/gitlab/click_house/migration.rake#L11

Otherwise, we just print a message.

But let's keep the feature flag for a few milestones, just in case our experimental-customers would want to disable it.

(I'm probably being too paranoid here)

See more in Run ClickHouse migrations during gitlab:db:conf... (!138661 - merged)

Related issues:

MR acceptance checklist

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

Edited by Vladimir Shushlin

Merge request reports

Loading