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

Add existingResourcePolicy restore CR validation to controller. #7757

Conversation

kaovilai
Copy link
Contributor

@kaovilai kaovilai commented Apr 29, 2024

Signed-off-by: Tiger Kaovilai tkaovila@redhat.com

Thank you for contributing to Velero!

Please add a summary of your change

Does your change fix a particular issue?

Fixes #7347

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.

@kaovilai kaovilai force-pushed the addExistingResourcePolicyRestoreCRValidation branch from f9b0d68 to 7cf879f Compare April 29, 2024 20:08
@kaovilai kaovilai force-pushed the addExistingResourcePolicyRestoreCRValidation branch 2 times, most recently from 857d280 to 8e1bcbf Compare April 29, 2024 20:09
sseago
sseago previously approved these changes Apr 29, 2024
anshulahuja98
anshulahuja98 previously approved these changes May 1, 2024
@anshulahuja98
Copy link
Collaborator

@kaovilai CI is failing, can you pl check
Verifying goimports - failed! Please run 'make update'.

Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com>
@kaovilai kaovilai dismissed stale reviews from anshulahuja98 and sseago via e1bef5b May 1, 2024 15:39
@kaovilai kaovilai force-pushed the addExistingResourcePolicyRestoreCRValidation branch from 8e1bcbf to e1bef5b Compare May 1, 2024 15:39
Copy link

codecov bot commented May 1, 2024

Codecov Report

Attention: Patch coverage is 85.71429% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 58.66%. Comparing base (8f78aaa) to head (3c937d4).
Report is 16 commits behind head on main.

Files Patch % Lines
pkg/cmd/cli/restore/create.go 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7757      +/-   ##
==========================================
+ Coverage   58.64%   58.66%   +0.01%     
==========================================
  Files         344      345       +1     
  Lines       28720    28733      +13     
==========================================
+ Hits        16843    16856      +13     
- Misses      10447    10448       +1     
+ Partials     1430     1429       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kaovilai
Copy link
Contributor Author

kaovilai commented May 1, 2024

Fixed make update! Thanks all.

sseago
sseago previously approved these changes May 1, 2024
anshulahuja98
anshulahuja98 previously approved these changes May 2, 2024
@kaovilai kaovilai dismissed stale reviews from anshulahuja98 and sseago via c304594 May 6, 2024 19:00
@kaovilai kaovilai force-pushed the addExistingResourcePolicyRestoreCRValidation branch from 1f2fa88 to c304594 Compare May 6, 2024 19:00
@kaovilai kaovilai force-pushed the addExistingResourcePolicyRestoreCRValidation branch from c304594 to c42683e Compare May 7, 2024 13:43
@kaovilai
Copy link
Contributor Author

kaovilai commented May 8, 2024

labels PR fix #7776

Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com>
@kaovilai kaovilai force-pushed the addExistingResourcePolicyRestoreCRValidation branch from c42683e to 3c937d4 Compare May 10, 2024 18:12
@reasonerjt reasonerjt merged commit 7e19cdb into vmware-tanzu:main May 14, 2024
66 checks passed
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.

existingResourcePolicy validation outside CLI
6 participants