Skip to content

Fix trigger condition for the cherry-pick workflow#8513

Closed
wangxin wants to merge 1 commit intosonic-net:masterfrom
wangxin:fix-workflow-trigger-condition
Closed

Fix trigger condition for the cherry-pick workflow#8513
wangxin wants to merge 1 commit intosonic-net:masterfrom
wangxin:fix-workflow-trigger-condition

Conversation

@wangxin
Copy link
Collaborator

@wangxin wangxin commented Jun 7, 2023

Description of PR

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911
  • 202012
  • 202205

Approach

What is the motivation for this PR?

PR #8511 updated the cherry-pick workflow to use lable "Request for 20xx branch". However, the workflow condition was not updated accordingly. The workflow actions are still skipped when labels like "Request for 20xx branch" is set.

How did you do it?

This change updated the trigger condition in the workflow yaml to match labels start with "Request for 20".

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

PR sonic-net#8511 updated the cherry-pick workflow to use lable "Request for 20xx branch".
However, the workflow condition was not updated accordingly. The workflow
actions are still skipped when labels like "Request for 20xx branch" is set.

This change updated the trigger condition in the workflow yaml to match labels
start with "Request for 20".

Signed-off-by: Xin Wang <[email protected]>
@wangxin
Copy link
Collaborator Author

wangxin commented Jun 8, 2023

Before auto cherry-pick a PR, it is better idea to have approval procedure. Let's follow the same practice of sonic-buildimage repo.

@wangxin wangxin closed this Jun 8, 2023
@wangxin wangxin deleted the fix-workflow-trigger-condition branch July 12, 2023 02:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant