Revisiting the Quarantining Process for E2E Tests

Looks like there are few gaps in the Quarantining Process that we follow, as a result of which, we tend to keep quarantining the tests and don't have a defined process to fix and de-quarantine the same.

Please find below the proposal to solve this issue.

Proposal

Existing Quarantined Tests

  • I’m planning to create an issue which has a checklist of all the quarantined test’s issues. (Similar to the unlabelled issues list for triaging) to fix the existing quarantined tests.
  • This list will be shared by the TAEs in Dev team (~17 tests)

Change in Quarantining/De-quarantining Process

  • We should have a triaging package which generates a new issue every monday which has the list of open quarantined test issues.
  • This issue should be assigned to the TAE in charge of “Failure Management Rotation” and it should be prioritized above any other tasks in his plate.
  • Within that week, the TAE is expected to debug, fix and dequarantine the test.
  • Whichever tests are quarantined in the current week will again be looked into by the next TAE in rotation and the process repeats.

@gl-quality Please share your thoughts.

Status

Quarantined Tests:

  • qa/specs/features/browser_ui/2_plan/issue/issue_suggestions_spec.rb
  • qa/specs/features/browser_ui/2_plan/issue/create_issue_spec.rb
  • qa/specs/features/browser_ui/2_plan/ee_epic/create_edit_delete_epic_spec.rb
  • qa/specs/features/browser_ui/2_plan/ee_epic/promote_issue_to_epic_spec.rb
  • qa/specs/features/api/2_plan/ee_epics_milestone_dates_spec.rb

  • qa/specs/features/browser_ui/7_configure/auto_devops/create_project_with_auto_devops_spec.rb

  • qa/specs/features/browser_ui/1_manage/project/import_github_repo_spec.rb
  • qa/specs/features/browser_ui/1_manage/login/login_via_oauth_spec.rb -> MR
  • qa/specs/features/browser_ui/1_manage/ee_group/group_ldap_sync_spec.rb
  • qa/specs/features/browser_ui/1_manage/project/ee_project_templates_spec.rb
  • browser_ui/1_manage/project/ee_project_templates_spec.rb

  • qa/specs/features/browser_ui/3_create/repository/add_file_template_spec.rb
  • qa/specs/features/browser_ui/3_create/repository/push_over_http_file_size_spec.rb
  • qa/specs/features/browser_ui/3_create/repository/protocol_v2_push_ssh_spec.rb
  • browser_ui/3_create/repository/protocol_v2_push_http_spec.rb
  • qa/specs/features/browser_ui/3_create/repository/add_list_delete_branches_spec.rb
  • qa/specs/features/browser_ui/3_create/repository/push_mirroring_over_http_spec.rb
  • qa/specs/features/browser_ui/3_create/repository/user_views_commit_diff_patch_spec.rb
  • qa/specs/features/browser_ui/3_create/web_ide/add_file_template_spec.rb
  • qa/specs/features/browser_ui/3_create/snippet/create_snippet_spec.rb
  • qa/specs/features/browser_ui/3_create/merge_request/create_merge_request_spec.rb
  • qa/specs/features/browser_ui/3_create/merge_request/merge_merge_request_from_fork_spec.rb
  • qa/specs/features/browser_ui/3_create/merge_request/squash_merge_request_spec.rb
  • qa/specs/features/browser_ui/3_create/merge_request/view_merge_request_diff_patch_spec.rb
  • qa/specs/features/browser_ui/3_create/repository/ee_assign_code_owners_spec.rb
  • browser_ui/3_create/repository/push_over_http_file_size_spec.rb
  • browser_ui/3_create/repository/user_views_commit_diff_patch_spec.rb
  • browser_ui/3_create/merge_request/ee_add_batch_comments_in_merge_request_spec.rb

  • qa/specs/features/browser_ui/6_release/deploy_key/add_deploy_key_spec.rb
  • qa/specs/features/browser_ui/6_release/deploy_key/clone_using_deploy_key_spec.rb

  • browser_ui/ee_geo/attachment_replication_spec.rb
Edited Jul 16, 2019 by Ramya Authappan
Assignee Loading
Time tracking Loading