Skip to content

BackfillWikiPermissionsInMainIndex integration test updates

Ravi Kumar requested to merge 426133-failure-in-ee-spec-elastic-migrate into master

What does this MR do and why?

I guess the issue is we are using wait_for_completion: true in the update_by_query to allow asynchronous tasks which is good for the performance of the migration. But sometimes the task is not completed during the test, and the test fails during assertion. We can put sleep to ensure the task is finished first. In some other tests, we are already using sleep

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

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 #426133 (closed)

Edited by Ravi Kumar

Merge request reports