Skip to content

Update test, remove from quarantine, and add checks and logging

Mark Lapierre requested to merge ml-fix-flaky-test into master

What does this MR do and why?

Investigation in #416821 (comment 1455901633) found that on rare occasions events weren't streamed, but the bug couldn't be reliably reproduced and eventually failed to reproduce at all, despite over 100 test runs.

This MR includes the additional checks that were added during the investigation, updates the test for recently event changes, and removes the test from quaratine.

How to set up and validate locally

From the qa directory:

bundle install
export QA_GITLAB_URL="http://gdk.test:3000" # Only needed if GDK is not running on http://127.0.0.1:3000
bundle exec rspec qa/specs/features/ee/api/10_govern/instance_audit_event_streaming_spec.rb

MR acceptance checklist

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

Edited by Mark Lapierre

Merge request reports