Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue 6734: spread backup pod evenly #6926

Merged
merged 2 commits into from Oct 10, 2023

Conversation

Lyndon-Li
Copy link
Contributor

Partially fix #6734, guide Kubernetes' scheduler to spread backup pods evenly across nodes as much as possible, so that data mover backup could achieve better parallelism

Signed-off-by: Lyndon-Li <lyonghui@vmware.com>
@codecov
Copy link

codecov bot commented Oct 8, 2023

Codecov Report

Merging #6926 (0a6c89a) into main (541425b) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #6926      +/-   ##
==========================================
+ Coverage   60.67%   60.69%   +0.02%     
==========================================
  Files         249      249              
  Lines       26476    26493      +17     
==========================================
+ Hits        16064    16081      +17     
  Misses       9268     9268              
  Partials     1144     1144              
Files Coverage Δ
pkg/exposer/csi_snapshot.go 81.06% <100.00%> (+1.13%) ⬆️

blackpiglet
blackpiglet previously approved these changes Oct 8, 2023
@mateusoliveira43
Copy link
Contributor

mateusoliveira43 commented Oct 9, 2023

Should the issue related to this PR be not closed after merge? (in description says it partially fixes it)

@mateusoliveira43
Copy link
Contributor

Can the changes related to codespell be removed from the PR?
The changes already got merged #6924 and there is a specific PR for it #6922

@ywk253100 ywk253100 merged commit 5961253 into vmware-tanzu:main Oct 10, 2023
24 checks passed
@weshayutin
Copy link
Contributor

weshayutin commented Oct 20, 2023

@sseago @shubham-pampattiwar can we propose for 1.12.1? Oh sorry.. perhaps it is #6935

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

Successfully merging this pull request may close these issues.

DataMover - datauploads and datadownloads resources aren't distributed equally among the workers.
5 participants