vmware-tanzu / velero

Backup and migrate Kubernetes applications and their persistent volumes
https://velero.io
Apache License 2.0
8.63k stars 1.39k forks source link

More prow actions #7784

Closed blackpiglet closed 4 months ago

blackpiglet commented 4 months ago

Thank you for contributing to Velero!

Please add a summary of your change

This PR introduces more k8s PROW style commands, including:

Also, add some introduction to the already supported PROW commands:

Please put those commands in the issue or the PR's comments to try them out.

Does your change fix a particular issue?

Fixes #7783

This PR is based on the ongoing PR #7776.

Please indicate you've done the following:

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 58.64%. Comparing base (f7c0244) to head (83c082f). Report is 2 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #7784 +/- ## ========================================== - Coverage 58.66% 58.64% -0.03% ========================================== Files 344 344 Lines 28731 28731 ========================================== - Hits 16854 16848 -6 - Misses 10448 10453 +5 - Partials 1429 1430 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

blackpiglet commented 4 months ago

The lgtm command is not included in this PR intentionally. Auto-merging seems too much for now. Let's get familiar with the PROW-style workflow first.

blackpiglet commented 4 months ago

/cc @kaovilai

blackpiglet commented 4 months ago

/cc @mmorel-35 @kaovilai @qiuming-best

kaovilai commented 4 months ago

Don't think cc worked for me

blackpiglet commented 4 months ago

/cc doesn't work for most cases. It only works for the maintainers' IDs listed in the collaborator list. It doesn't even work for those in the collaborator list through an alias.