Skip to content

Fix N+1 test of pipeline seed

Furkan Ayhan requested to merge fix-sql-n1-test-for-pipeline-seed into master

What does this MR do?

This MR adds a warm-up stage for N+1 test.

We need it for checking the actual result because there are some non-related SQL queries like;

  • InternalId Load
  • InternalId Update
  • Ci::Ref Load

And those queries may lead us to false positives. The first control state always has SQL queries more than it should.


I run into this after opening this MR !60805 (closed). Then realized that there is something wrong with the test.

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 Furkan Ayhan

Merge request reports