Skip to content

Remove cloud_license_enabled database column

What does this MR do?

Part of https://gitlab.com/gitlab-org/gitlab/-/issues/335811

After previously removing any references to the application setting's cloud_license_enabled and ignoring the column in the model, the database column can now be removed.

Up migration:

== 20210706112800 RemoveCloudLicenseEnabledFromApplicationSettings: migrating =
-- remove_column(:application_settings, :cloud_license_enabled)
   -> 0.0024s
== 20210706112800 RemoveCloudLicenseEnabledFromApplicationSettings: migrated (0.0092s) 

Down migration:

== 20210706112800 RemoveCloudLicenseEnabledFromApplicationSettings: reverting =
-- add_column(:application_settings, :cloud_license_enabled, :boolean, {:null=>false, :default=>false})
   -> 0.0030s
== 20210706112800 RemoveCloudLicenseEnabledFromApplicationSettings: reverted (0.0097s) 

Screenshots or Screencasts (strongly suggested)

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
Edited by Corinna Gogolok

Merge request reports