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

Fix client wait problem after async operation change #5964

Merged
merged 1 commit into from
Mar 8, 2023

Conversation

Lyndon-Li
Copy link
Contributor

Fix client wait problem after async operation change, velero backup/restore --wait should check a full list of the terminal status

Signed-off-by: Lyndon-Li <[email protected]>
@Lyndon-Li Lyndon-Li force-pushed the fix-client-wait-problem branch from 25f9d55 to 28d636b Compare March 8, 2023 01:29
@codecov-commenter
Copy link

codecov-commenter commented Mar 8, 2023

Codecov Report

Merging #5964 (25f9d55) into main (8bed159) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

❗ Current head 25f9d55 differs from pull request most recent head 28d636b. Consider uploading reports for the commit 28d636b to get more accurate results

@@            Coverage Diff             @@
##             main    #5964      +/-   ##
==========================================
- Coverage   39.76%   39.75%   -0.01%     
==========================================
  Files         256      256              
  Lines       23236    23237       +1     
==========================================
  Hits         9239     9239              
- Misses      13299    13300       +1     
  Partials      698      698              
Impacted Files Coverage Δ
pkg/cmd/cli/backup/create.go 22.47% <0.00%> (-0.11%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Lyndon-Li Lyndon-Li merged commit 2dab344 into vmware-tanzu:main Mar 8, 2023
@Lyndon-Li Lyndon-Li deleted the fix-client-wait-problem branch March 8, 2023 03:06
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.

4 participants