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

[.github/scripts/auto-backport.py] improve label logic #542

Merged
merged 1 commit into from
Sep 19, 2024

Conversation

yaronkaikov
Copy link
Collaborator

@yaronkaikov yaronkaikov commented Sep 18, 2024

In this commit there few improvments to the backport automation process based on the work done in #541

  1. Create new parameter --commit to be use only when --pull-request is provided
  2. Trigger workflow when label is backport/x.y only, rather then backport/ as it was defined before. This way we will not trigger to flow when backport/x.y-done was added
  3. when running on specific PR with --pull-request make sure we don't pass an empty commit
  4. fix get_prs_from_commits() while trying to do get_commit on the wrong sha1 value

In this commit there few improvments to the backport automation process
based on the work done in scylladb#541

1. Create new parameter `--commit` to be use only when `--pull-request`
   is provided
2. Trigger workflow when label is `backport/x.y` only, rather then
   `backport/` as it was defined before. This way we will not trigger to
flow when `backport/x.y-done` was added
3. when running on specific PR with `--pull-request` make sure we don't
   pass an empty commit
4. fix `get_prs_from_commits()` while trying to do `get_commit` on the
   wrong sha1 value
@fruch
Copy link
Collaborator

fruch commented Sep 19, 2024

@yaronkaikov are you gonna apply those on dtest/sct as well ?

@yaronkaikov
Copy link
Collaborator Author

@yaronkaikov are you gonna apply those on dtest/sct as well ?

yep, soon

@yaronkaikov yaronkaikov merged commit d465204 into scylladb:next Sep 19, 2024
1 check passed
@yaronkaikov yaronkaikov deleted the sync-automation branch September 19, 2024 07:01
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