Skip to content

Add members to Rubocop UpdateLargeTable list

Etienne Baqué requested to merge 336994-add-members-to-rubocop-migrations into master

What does this MR do?

Closes #336994 (closed)

This small MR adds the members table to the Rubocop UpdateLargeTable list.

How to setup and validate locally (strongly suggested)

In rubocop/rubocop-migrations.yml:17, a comment points to the fact that tables with size >= 10Gb should be in that list. Checking size in production:

gitlabhq_production=> SELECT pg_size_pretty(pg_total_relation_size('members'));
 pg_size_pretty 
----------------
 11 GB
(1 row)

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 #336994 (closed)

Edited by Etienne Baqué

Merge request reports