Skip to content

feat(backoff): add initial delay support for backoff operations#860

Merged
BSWANG merged 1 commit intoAliyunContainerService:mainfrom
l1b0k:feat/backoff
Jul 28, 2025
Merged

feat(backoff): add initial delay support for backoff operations#860
BSWANG merged 1 commit intoAliyunContainerService:mainfrom
l1b0k:feat/backoff

Conversation

@l1b0k
Copy link
Collaborator

@l1b0k l1b0k commented Jul 28, 2025

  • Introduce ExtendedBackoff struct to support initial delay
  • Update backoff map to use ExtendedBackoff instead of wait.Backoff
  • Modify ExponentialBackoffWithContext to include initial delay
  • Update affected packages to use the new ExtendedBackoff structure
  • Add tests for the new initial delay functionality

@l1b0k l1b0k force-pushed the feat/backoff branch 3 times, most recently from 513d5de to b4224dc Compare July 28, 2025 11:02
- Introduce ExtendedBackoff struct to support initial delay
- Update backoff map to use ExtendedBackoff instead of wait.Backoff
- Modify ExponentialBackoffWithContext to include initial delay
- Update affected packages to use the new ExtendedBackoff structure
- Add tests for the new initial delay functionality

Signed-off-by: l1b0k <libokang.lbk@alibaba-inc.com>
@codecov
Copy link

codecov bot commented Jul 28, 2025

Codecov Report

❌ Patch coverage is 75.86207% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 35.66%. Comparing base (bff2c6b) to head (aa3ebe5).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
pkg/controller/eni/eni.go 35.71% 5 Missing and 4 partials ⚠️
pkg/backoff/backoff.go 86.95% 2 Missing and 1 partial ⚠️
pkg/controller/pod/pod_controller.go 50.00% 0 Missing and 1 partial ⚠️
pkg/eni/crdv2.go 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #860      +/-   ##
==========================================
+ Coverage   35.58%   35.66%   +0.07%     
==========================================
  Files         130      130              
  Lines       18890    18919      +29     
==========================================
+ Hits         6722     6747      +25     
  Misses      11413    11413              
- Partials      755      759       +4     
Flag Coverage Δ
unittests 35.66% <75.86%> (+0.07%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@BSWANG BSWANG merged commit 94ffaf4 into AliyunContainerService:main Jul 28, 2025
9 checks passed
@l1b0k l1b0k deleted the feat/backoff branch July 28, 2025 12:07
cursor bot pushed a commit that referenced this pull request Sep 7, 2025
feat(backoff): add initial delay support for backoff operations
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