Cells: Iteration 2: Configure `namespaces` adjacent tables to be `gitlab_main_cell` to support `User can create Group`
Problem
As part of the #404762 (comment 1342204543)+ a set of tables were identified as belong to cell. Those tables were touched when trying to execute workflow User can create group.
Proposal
- Assign all below tables to
gitlab_main_cell
. - Identify all cross-joins violations and define which ones should be solved to support this workflow.
Tables
- group_crm_settings
- group_features
- group_group_links
- group_import_states
- cluster_enabled_grants - unsure?
- dependency_proxy_group_settings
- namespace_details
- namespace_settings
- projects - probably cell-local
- project_features
- project_authorizations
- merge_requests
- issues
- members
- web_hooks - problematic, does it have instance-level system hooks?
- integrations - problematic, since it also has instance-level templates
Adding to the above list
The following tables come from the discussion in #415201 (comment 1495061091)
-
chat_teams
- belongs to namespace_id -
user_group_callouts
- because it belongs to group_id -
saml_providers
- belongs to namespace_id -
upcoming_reconciliations
- belongs to namespace_id -
gitlab_subscriptions
- belongs to namespace_id notification_settings
Update on Progress:
We are attempting to move tables in multiple small MRs rather than 1 big MR.
- With !128639 (merged) we moved the following:
- chat_teams
- group_crm_settings
- group_features
- group_group_links
- group_import_states
- With !128709 (merged), we moved the following
- cluster_enabled_grants
- dependency_proxy_group_settings
- namespace_details
- namespace_settings
- user_group_callouts
- With !129304 (merged)
- upcoming_reconciliations
- With !129173 (merged)
- projects
- With !129303 (merged)
- saml_providers
- With !129340 (merged)
- gitlab_subscriptions
- With !129352 (merged)
- project_features
- With !129365 (merged)
- project_authorizations
- With !129394 (merged)
- merge_requests
- With !129452 (merged)
- issues
- With !130038 (merged)
- notification_settings
- With !130032 (merged)
- integrations
- With !130039 (merged)
- web_hooks
- With !129287 (merged)
- members
Follow-up epic: notfication_settings, web_hooks and integrations tables need to be split: &11302 (closed)
Edited by Manoj M J