Skip to content

Conversation

@ywk253100
Copy link
Contributor

Restore finalizer and managedFields of metadata during the restoration

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.

Restore finalizer and managedFields of metadata during the restoration

Signed-off-by: Wenkai Yin(尹文开) <[email protected]>
@ywk253100 ywk253100 changed the title Restore finalizer and managedFields [cherry-pick]Restore finalizer and managedFields Feb 20, 2023
@codecov-commenter
Copy link

codecov-commenter commented Feb 20, 2023

Codecov Report

Merging #5877 (f0336e3) into release-1.10 (42fb499) will decrease coverage by 0.01%.
The diff coverage is 52.54%.

@@               Coverage Diff                @@
##           release-1.10    #5877      +/-   ##
================================================
- Coverage         40.95%   40.95%   -0.01%     
================================================
  Files               240      240              
  Lines             20715    20762      +47     
================================================
+ Hits               8484     8503      +19     
- Misses            11611    11632      +21     
- Partials            620      627       +7     
Impacted Files Coverage Δ
pkg/builder/object_meta.go 0.00% <0.00%> (ø)
pkg/restore/restore.go 64.17% <25.00%> (-0.49%) ⬇️
pkg/restore/service_action.go 60.36% <69.23%> (-0.62%) ⬇️

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

@ywk253100 ywk253100 merged commit 4512160 into vmware-tanzu:release-1.10 Feb 20, 2023
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