Skip to content

Draft: Remove two flaky examples from reliable

Anastasia McDonald requested to merge a_mcdonald-master-patch-e84b into master

What does this MR do and why?

Removes two flaky examples that were recently promoted from reliable. In both cases it was a repeated intermittent failure from the past that happened within a week of promotion so it doesn't justify the risk of keeping the tests in reliable bucket and blocking pipelines. See gitlab-org/quality/testcases#3335 (closed) and gitlab-org/quality/testcases#2226 (closed).

Affected examples:

  • /qa/qa/specs/features/ee/browser_ui/3_create/repository/file_locking_spec.rb | Create File Locking locks a directory and tries to push as a second user
  • /qa/qa/specs/features/api/3_create/repository/files_spec.rb | Create API basics user creates a project with a file and deletes them afterwards

Also removes raw file access sets no-cache headers as expected from E2Es altogether as it is sufficiently covered at the lower level with https://gitlab.com/gitlab-org/gitlab/-/blob/368a4c8a002ab27fe35233da056a65c0c91144ff/spec/requests/api/files_spec.rb#L852.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports