Skip to content

test: add LRO for resource clean job#16071

Merged
neil-yechenwei merged 2 commits into
devfrom
neil/fixcleanjob02
Jun 2, 2026
Merged

test: add LRO for resource clean job#16071
neil-yechenwei merged 2 commits into
devfrom
neil/fixcleanjob02

Conversation

@neil-yechenwei

@neil-yechenwei neil-yechenwei commented Jun 1, 2026

Copy link
Copy Markdown
Contributor
  1. Updated the execute request logic to support real retries (up to 6 retry attempts).
  2. Retries are only applied to recoverable errors: 429, 408, and 5xx.
  3. Added exponential backoff (starting at 2s, capped at 60s) plus random jitter to prevent synchronized retry spikes under concurrency.
  4. Prioritizes server-provided throttling headers when present:
  • x-ms-retry-after-ms
  • retry-after
  1. Non-recoverable errors still fail fast, without blind retries.
  2. Update the trigger time for rerun.yml

Note: After the changes are applied, the job will be completed within 30 minutes.

@github-actions

github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

E2E Test Selection — AI Selected

Why these tests: .github/workflows/rerun.yml is infrastructure-only (rule 8) and packages/tests/src/utils/cleanHelper.ts is used exclusively by ui-test files, not by any of the available e2e test cases.

Cases selected (0):


View pipeline run

Need to run more tests?

Comment on this PR:

  • /e2e-run ./path/to/test.tests.ts — add specific cases to AI selection
  • /e2e-run-all — run all e2e cases

Then re-run the workflow.

@codecov

codecov Bot commented Jun 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.36%. Comparing base (eb0bad7) to head (0184771).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##              dev   #16071   +/-   ##
=======================================
  Coverage   93.36%   93.36%           
=======================================
  Files         597      597           
  Lines       37443    37443           
  Branches     6993     6968   -25     
=======================================
  Hits        34957    34957           
  Misses       1836     1836           
  Partials      650      650           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@neil-yechenwei neil-yechenwei merged commit 7e795c3 into dev Jun 2, 2026
86 of 98 checks passed
@neil-yechenwei neil-yechenwei deleted the neil/fixcleanjob02 branch June 2, 2026 01:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants