Skip to content

feat: add rollout support for LeaderWorkerSet#317

Open
kirat10x wants to merge 1 commit intoopenkruise:masterfrom
kirat10x:lws
Open

feat: add rollout support for LeaderWorkerSet#317
kirat10x wants to merge 1 commit intoopenkruise:masterfrom
kirat10x:lws

Conversation

@kirat10x
Copy link

@kirat10x kirat10x commented Nov 1, 2025

Ⅰ. Describe what this PR does

Ⅱ. Does this pull request fix one issue?

Ⅲ. Special notes for reviews

Signed-off-by: Kirat <kiratdal1@gmail.com>
@kruise-bot kruise-bot requested review from FillZpp and veophi November 1, 2025 15:19
@kruise-bot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign zmberg for approval by writing /assign @zmberg in a comment. For more information see:The Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@codecov
Copy link

codecov bot commented Nov 1, 2025

Codecov Report

❌ Patch coverage is 1.72414% with 57 lines in your changes missing coverage. Please review.
✅ Project coverage is 50.26%. Comparing base (501d832) to head (47f3ba6).
⚠️ Report is 4 commits behind head on master.

Files with missing lines Patch % Lines
pkg/util/controller_finder.go 0.00% 50 Missing ⚠️
pkg/util/workloads_utils.go 0.00% 4 Missing ⚠️
...g/controller/batchrelease/batchrelease_executor.go 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #317      +/-   ##
==========================================
- Coverage   50.60%   50.26%   -0.34%     
==========================================
  Files          64       64              
  Lines        8201     8258      +57     
==========================================
+ Hits         4150     4151       +1     
- Misses       3478     3534      +56     
  Partials      573      573              
Flag Coverage Δ
unittests 50.26% <1.72%> (-0.34%) ⬇️

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants