Skip to content

Fix compliance framework migration on CE

Adam Hegyi requested to merge fix-ee-compliance-inline-data-migration-on-ce into master

What does this MR do?

Related issue: #272624 (comment 437845087)

TLDR: We must run on both CE and EE the inline data migrations. In some cases constraints are depending on the changed data.

In this particular case the NOT NULL check constraint was depending on a previous data migration which was made EE only. This caused error when a downgraded instance (from EE to CE) was running the migrations.

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • 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 Adam Hegyi

Merge request reports