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

Restore finalizer and managedFields #5808

Merged
merged 1 commit into from
Feb 6, 2023

Conversation

ywk253100
Copy link
Contributor

Restore finalizer and managedFields of metadata during the restoration

Signed-off-by: Wenkai Yin(尹文开) [email protected]

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
Copy link
Contributor Author

Related issues: #5701 #5580

Restore finalizer and managedFields of metadata during the restoration

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

codecov-commenter commented Jan 31, 2023

Codecov Report

Merging #5808 (60be8e5) into release-1.9 (dd66088) will increase coverage by 0.01%.
The diff coverage is 20.00%.

@@               Coverage Diff               @@
##           release-1.9    #5808      +/-   ##
===============================================
+ Coverage        41.62%   41.63%   +0.01%     
===============================================
  Files              214      214              
  Lines            18599    18617      +18     
===============================================
+ Hits              7741     7751      +10     
- Misses           10274    10281       +7     
- Partials           584      585       +1     
Impacted Files Coverage Δ
pkg/builder/object_meta.go 0.00% <0.00%> (ø)
pkg/restore/restore.go 63.91% <25.00%> (+0.07%) ⬆️

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

@ywk253100 ywk253100 requested a review from blackpiglet February 2, 2023 01:27
@ywk253100 ywk253100 merged commit 0f063c4 into vmware-tanzu:release-1.9 Feb 6, 2023
ywk253100 added a commit to ywk253100/velero that referenced this pull request Feb 6, 2023
Restore finalizer and managedFields of metadata during the restoration

Signed-off-by: Wenkai Yin(尹文开) <[email protected]>
ywk253100 added a commit to ywk253100/velero that referenced this pull request Feb 6, 2023
Restore finalizer and managedFields of metadata during the restoration

Signed-off-by: Wenkai Yin(尹文开) <[email protected]>
ywk253100 added a commit to ywk253100/velero that referenced this pull request Feb 6, 2023
Restore finalizer and managedFields of metadata during the restoration

Signed-off-by: Wenkai Yin(尹文开) <[email protected]>
ywk253100 added a commit to ywk253100/velero that referenced this pull request Feb 8, 2023
ywk253100 added a commit to ywk253100/velero that referenced this pull request Feb 8, 2023
This reverts commit 0f063c4.

Signed-off-by: Wenkai Yin(尹文开) <[email protected]>
ywk253100 added a commit to ywk253100/velero that referenced this pull request Feb 9, 2023
Restore finalizer and managedFields of metadata during the restoration

Signed-off-by: Wenkai Yin(尹文开) <[email protected]>
ywk253100 added a commit that referenced this pull request Feb 9, 2023
This reverts commit 0f063c4.

Signed-off-by: Wenkai Yin(尹文开) <[email protected]>
ywk253100 added a commit to ywk253100/velero that referenced this pull request Feb 9, 2023
Restore finalizer and managedFields of metadata during the restoration

Signed-off-by: Wenkai Yin(尹文开) <[email protected]>
ywk253100 added a commit to ywk253100/velero that referenced this pull request Feb 13, 2023
Restore finalizer and managedFields of metadata during the restoration

Signed-off-by: Wenkai Yin(尹文开) <[email protected]>
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.

5 participants