-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
feat: enable cherry-pick bot #14151
feat: enable cherry-pick bot #14151
Conversation
Signed-off-by: Alan Clucas <[email protected]>
/test
commentThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a great addition. Thanks!
/cherry-pick release-3.6 |
Cherry-pick failed with |
/cherry-pick release-3.5 |
Cherry-pick failed with |
(cherry picked from commit e4f51c2)
(cherry picked from commit e4f51c2) Signed-off-by: Alan Clucas <[email protected]>
The |
@Joibel Why does my normal comments also trigger CI: https://github.com/argoproj/argo-workflows/actions/runs/13213864653? |
I forgot there were two entrypoints to the CI workflow, and only added an I'll work out whether to remove |
Partial revert of #14151 Signed-off-by: Alan Clucas <[email protected]>
#14164 is a partial revert to fix these two issues. |
Part of #12592
Motivation
Maintaining released versions of argo-workflows is not automated. This will help by allowing the backporting of commits as they are merged rather than by the release manager at the time of release.
Each PR can be CI tested independently then, and only those that pass get merged.
Modifications
Enabled CI on release branch PRs.
Added a new /cherry-pick action, using the same cherry-pick-bot as argo-cd already use. I have already enabled that bot on this project, but it needs a configuration file as provided here.
Verification
This mostly needs testing after merging.
Documentation
Added documentation for how to use
/cherry-pick
, and for the other comment based command which already existed:/retest