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

Create the backup repository only when it doesn't exist #6884

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

ywk253100
Copy link
Contributor

When preparing a backup repository, Velero tries to connect to it, if fails then create it. The repository status always records the error reported by creation but the real reason maybe caused by the connect operation. This is confuseing and hard to debug

Thank you for contributing to Velero!

Please add a summary of your change

Does your change fix a particular issue?

Fixes #(issue)

Please indicate you've done the following:

  • Accepted the DCO. Commits without the DCO will delay acceptance.
  • Created a changelog file or added /kind changelog-not-required as a comment on this pull request.
  • Updated the corresponding documentation in site/content/docs/main.

@ywk253100 ywk253100 added the kind/changelog-not-required PR does not require a user changelog. Often for docs, website, or build changes label Sep 28, 2023
@ywk253100 ywk253100 marked this pull request as ready for review September 28, 2023 02:43
Lyndon-Li
Lyndon-Li previously approved these changes Sep 28, 2023
qiuming-best
qiuming-best previously approved these changes Sep 28, 2023
When preparing a backup repository, Velero tries to connect to it, if fails then create it. The repository status always records the error reported by creation but the real reason maybe caused by the connect operation. This is confuseing and hard to debug

Signed-off-by: Wenkai Yin(尹文开) <[email protected]>
@codecov
Copy link

codecov bot commented Sep 28, 2023

Codecov Report

Merging #6884 (61a6c1b) into main (73ea00b) will increase coverage by 0.01%.
Report is 2 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #6884      +/-   ##
==========================================
+ Coverage   60.75%   60.76%   +0.01%     
==========================================
  Files         247      247              
  Lines       26314    26322       +8     
==========================================
+ Hits        15987    15995       +8     
  Misses       9190     9190              
  Partials     1137     1137              
Files Coverage Δ
pkg/repository/provider/unified_repo.go 87.43% <100.00%> (+0.03%) ⬆️

... and 1 file with indirect coverage changes

@qiuming-best qiuming-best merged commit c619179 into vmware-tanzu:main Sep 28, 2023
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has-unit-tests kind/changelog-not-required PR does not require a user changelog. Often for docs, website, or build changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants