velero icon indicating copy to clipboard operation
velero copied to clipboard

RestoreItemAction v2 design

Open sseago opened this issue 2 years ago • 2 comments

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 avatar Sep 28 '22 18:09 sseago

/kind changelog-not-required

sseago avatar Sep 28 '22 18:09 sseago

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%) :arrow_up:
pkg/repository/udmrepo/kopialib/backend/utils.go 70.73% <100.00%> (+0.73%) :arrow_up:

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

codecov-commenter avatar Sep 28 '22 18:09 codecov-commenter

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

sseago avatar Dec 06 '22 18:12 sseago

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

weshayutin avatar Dec 06 '22 21:12 weshayutin