Skip to content

Add user_cap column to namespace_settings

What does this MR do?

Relates to #332590 (closed). Epic, for more context, is here.

This MR adds a user_cap integer column to the namespace_settings table.

As written in this epic, this is a continuation of something that was done for self-managed, as we're bringing that user cap feature to Saas. This is the equivalent MR to this one here done for self-managed (ie. the MR where the user cap column is created for self-managed instances)

Migration Up/Down

etienne@dell ~/src/gdk/gitlab(332590-add-user-cap-column-to-namespace-settings ✗) rails db:migrate:up VERSION=20210705132928
== 20210705132928 AddNewUserSignupsCapToNamespaceSettings: migrating ==========
-- add_column(:namespace_settings, :new_user_signups_cap, :integer, {:null=>true})
   -> 0.0008s
== 20210705132928 AddNewUserSignupsCapToNamespaceSettings: migrated (0.0042s) =

etienne@dell ~/src/gdk/gitlab(332590-add-user-cap-column-to-namespace-settings ✗) rails db:migrate:down VERSION=20210705132928
== 20210705132928 AddNewUserSignupsCapToNamespaceSettings: reverting ==========
-- remove_column(:namespace_settings, :new_user_signups_cap)
   -> 0.0007s
== 20210705132928 AddNewUserSignupsCapToNamespaceSettings: reverted (0.0044s) =

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Related to #332590 (closed)

Edited by Etienne Baqué

Merge request reports