Skip to content

Fix schema order for published incidents

Sarah Yasonik requested to merge sy-fix-schema-order into master

What does this MR do?

!30120 (closed) merged a migration which had several changes during review, including reordering columns. This got missed in db/structure.sql, so now db/structure.sql unfortunately has out-of-order columns relative to the migration.

This MR updates db/structure.sql to reflect the actual migration. See !29994 (comment 333453951).

Changes via:

$ rake db:migrate:down VERSION=20200416005331
== 20200416005331 CreateStatusPagePublishedIncidents: reverting ===============
-- drop_table(:status_page_published_incidents)
   -> 0.0027s
== 20200416005331 CreateStatusPagePublishedIncidents: reverted (0.0028s) ======

$ rake db:migrate
== 20200416005331 CreateStatusPagePublishedIncidents: migrating ===============
-- create_table(:status_page_published_incidents)
   -> 0.0105s
== 20200416005331 CreateStatusPagePublishedIncidents: migrated (0.0216s) ======

Screenshots

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 Mayra Cabrera

Merge request reports