Skip to content
Snippets Groups Projects

Draft: Revert "Organization contains namespaces"

Closed Alex Pooley requested to merge 412771-revert-organization-contains-namespaces into master
1 unresolved thread

What does this MR do and why?

Organization contains namespaces (!120822 - merged) caused a deployment problem. We are reverting the MR to avoid chance of deployment problems for self managed customers.

Further to the MR revert

  • Remove the namespaces.organization_id column if it exists
  • No longer track Organization record deletions
  • unprepare_async_index the namespaces.organization_id index

MR acceptance checklist

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

Closes #412771 (closed)

Edited by Alex Pooley

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
1 # frozen_string_literal: true
2
3 class RemoveOrganizationIdOnNamespaces < Gitlab::Database::Migration[2.1]
4 DEFAULT_ORGANIZATION_ID = 1
5
6 enable_lock_retries!
7
8 def up
9 remove_column :namespaces, :organization_id, if_exists: true
  • mentioned in issue #412771 (closed)

  • Author Maintainer

    Closing this MR in favor of !122809 (merged)

  • closed

  • Alex Pooley mentioned in merge request !122809 (merged)

    mentioned in merge request !122809 (merged)

  • Please register or sign in to reply
    Loading