Skip to content

Use dynamic top-level group for restrict_by_ip_address_spec

Sanad Liaquat requested to merge qa-shl-restrict-ip-address-spec-logging into master

What does this MR do?

Switches to using a dynamic top-level group and enables Gold plan on it on when running against .com environments - Adds more logging and assertions to aid in further investigation.

I am keeping the test in quarantine until we have more data to further investigate the issue.

Related issue: #212544 (closed)

The issue causing the flakiness is either

  1. Two instances of the tests running at the same time and both changing the ip address of gitlab-qa-ip-restricted-sandbox-group => Remedy: Create a group dynamically and enable Gold Plan
  2. The ip address is not correctly updated => Remedy: Fail the test if this is the case and fix the cause in next iteration before un-quarantining.

Conformity

Edited by Sanad Liaquat

Merge request reports