Skip to content

Add delay for report ingestion

Jonathan Schafer requested to merge 297164-add-delay-for-report-ingestion into master

What does this MR do and why?

This MR adds a delay after the pipeline runs in the security_reports_spec.rb end-to-end test. It takes a variable amount of time to ingest the report. After multiple tests, 2 minutes seems to be enough time for the report to ingest completely before running the rest of the test.

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

I ran a batch script to run the test 100 times, and was able to run it with a sleep 120 to get 0 failures. I also tried 15, 30, and 60.

#!/bin/bash

for i in {1..100}
do
  bundle exec bin/qa Test::Instance::All http://172.16.123.1:3000 -- qa/specs/features/ee/browser_ui/secure/security_reports_spec.rb | grep "5 examples\|Finished in" >> ~/qa.out
  echo .
done

echo "ALL DONE"

MR acceptance checklist

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

Related to #297164 (closed)

Edited by Jonathan Schafer

Merge request reports