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

RestoreItemAction v2 design #5408

Merged
merged 1 commit into from
Dec 12, 2022

Conversation

sseago
Copy link
Collaborator

@sseago sseago commented Sep 28, 2022

Signed-off-by: Scott Seago [email protected]

This includes necessary changes to support async item action monitoring as well as the Wait for Additional Items feature.

This includes the design for the RestoreItemAction v2 API which is to be implemented in the Velero 1.11 dev cycle.

Thank you for contributing to Velero!

Does your change fix a particular issue?

Fixes #(issue)

Please indicate you've done the following:

  • [x ] Accepted the DCO. Commits without the DCO will delay acceptance.
  • [x ] 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.

@sseago
Copy link
Collaborator Author

sseago commented Sep 28, 2022

/kind changelog-not-required

@github-actions github-actions bot added the Area/Design Design Documents label Sep 28, 2022
@github-actions github-actions bot added the kind/changelog-not-required PR does not require a user changelog. Often for docs, website, or build changes label Sep 28, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #5408 (a4e8d05) into main (66f6365) will increase coverage by 0.02%.
The diff coverage is 84.21%.

@@            Coverage Diff             @@
##             main    #5408      +/-   ##
==========================================
+ Coverage   40.56%   40.59%   +0.02%     
==========================================
  Files         236      236              
  Lines       20385    20399      +14     
==========================================
+ Hits         8269     8280      +11     
- Misses      11511    11513       +2     
- Partials      605      606       +1     
Impacted Files Coverage Δ
pkg/repository/provider/unified_repo.go 57.83% <76.92%> (+0.77%) ⬆️
pkg/repository/udmrepo/kopialib/backend/utils.go 70.73% <100.00%> (+0.73%) ⬆️

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

Signed-off-by: Scott Seago <[email protected]>
@sseago
Copy link
Collaborator Author

sseago commented Dec 6, 2022

Now that the overall design changes have been approved, this PR is ready for review.

@weshayutin
Copy link
Contributor

@dsu-igeek @ywk253100 @blackpiglet @qiuming-best @Lyndon-Li @reasonerjt Please review as time permits, thank you!

@shubham-pampattiwar shubham-pampattiwar merged commit 30b7ed8 into vmware-tanzu:main Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area/Design Design Documents area/progress-monitoring 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.

6 participants