Adds retry attempts to give pipeline status widget time to update

What does this MR do and why?

Describe in detail what your merge request does and why.

This adds retry attempts to give the pipeline status widget time to show that it has already passed.

Related: #340878 (closed)

MR acceptance checklist

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

• I have evaluated the MR acceptance checklist for this MR.

changed milestone to %14.4

assigned to @richard.chong

marked the checklist item I have evaluated the MR acceptance checklist for this MR. as completed

added QA label

	1 Warning
	Please add a merge request type to this merge request.

	1 Message
	CHANGELOG missing: If you want to create a changelog entry for GitLab FOSS, add the Changelog trailer to the commit message you want to add to the changelog. If you want to create a changelog entry for GitLab EE, also add the EE: true trailer to your commit message. If this merge request doesn't need a CHANGELOG entry, feel free to ignore this message.

Reviewer roulette

Changes that require review have been detected!

Please refer to the table below for assigning reviewers and maintainers suggested by Danger in the specified category:

Category	Reviewer	Maintainer
QA	Will Meek (@willmeek) (UTC+1, 8 hours ahead of @richard.chong)	Andrejs Cunskis (@acunskis) (UTC+3, 10 hours ahead of @richard.chong)

To spread load more evenly across eligible reviewers, Danger has picked a candidate for each review slot, based on their timezone. Feel free to override these selections if you think someone else would be better-suited or use the GitLab Review Workload Dashboard to find other available reviewers.

To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines. Please consider assigning a reviewer or maintainer who is a domain expert in the area of the merge request.

Once you've decided who will review this merge request, assign them as a reviewer! Danger does not automatically notify them for you.

If needed, you can retry the danger-review job that generated this comment.

Generated by Danger

Allure report

allure-report-publisher generated test report for 2b337353!

review-qa-smoke: test report
review-qa-all: test report
package-and-qa: test report

requested review from @willmeek

mentioned in issue gitlab-org/quality/pipeline-triage#100 (closed)

Setting label(s) Quality based on QA.

added Quality label

added 1 commit

• 5fbc49c7 - Doubles the max_duration time to wait for pipeline status widget

Compare with previous version

added 731 commits

• 5fbc49c7...bb3fe4a1 - 729 commits from branch master
• f0e2fb67 - Adds retry attempts to give pipeline status widget time to update
• 1f3af252 - Doubles the max_duration time to wait for pipeline status widget

Compare with previous version

added 18 commits

• 1f3af252...7928d956 - 16 commits from branch master
• e0e687a5 - Adds retry attempts to give pipeline status widget time to update
• 2b337353 - Doubles the max_duration time to wait for pipeline status widget

Compare with previous version

The package-and-qa job from pipeline https://gitlab.com/gitlab-org/gitlab/-/pipelines/379231208 triggered https://gitlab.com/gitlab-org/build/omnibus-gitlab-mirror/-/pipelines/379254386 downstream.

The qa-test job from pipeline https://gitlab.com/gitlab-org/build/omnibus-gitlab-mirror/-/pipelines/379254386 triggered https://gitlab.com/gitlab-org/gitlab-qa-mirror/-/pipelines/379282324 downstream.

Screenshot from allure report warms my heart

The gitlab-qa downstream pipeline failed! .

Both failures look unrelated. I retried them to see if they'll pass.

Expand for screenshot

• #300673 (closed)

Expand for screenshot

• #299023 (closed)

And... They passed!

approved this merge request

Passing to @acunskis for maintainer review

removed review request for @willmeek

@willmeek, thanks for approving this merge request.

This is the first time the merge request is approved. To ensure full test coverage, a new pipeline has been started.

For more info, please refer to the following links:

• rspec minimal jobs
• jest minimal jobs
• merging a merge request.

resolved all threads

requested review from @acunskis

Great change! It is a very good practice to not wrap expectation with retrier as it will create far better exception messages.

approved this merge request

enabled an automatic merge when the pipeline for 52b06535 succeeds

canceled the automatic merge

resolved all threads

merged

mentioned in commit a1297233

added workflowstaging-canary label

added workflowstaging label and removed workflowstaging-canary label

added workflowcanary label and removed workflowstaging label

added workflowproduction label and removed workflowcanary label

added releasedcandidate label

mentioned in issue #340878 (closed)

mentioned in issue #337595 (closed)

added releasedpublished label and removed releasedcandidate label