Skip to content

[action] [PR:23101] feat: add cancel_pr for testplan.py#23116

Merged
mssonicbld merged 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/23101
Mar 23, 2026
Merged

[action] [PR:23101] feat: add cancel_pr for testplan.py#23116
mssonicbld merged 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/23101

Conversation

@mssonicbld
Copy link
Collaborator

Description of PR

Summary: Adding the ability to cancel previous PR to integrate with #22861
Fixes # (issue) 37065882

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
  • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

Approach

What is the motivation for this PR?

Currently, when running a new testplan, there is no safety check to cancel all the previous dangling job.

This add the ability to test_plan.py to cancel all related job with PR number using cancel_pr. However this PR does not add the call, only the implementation. The real call will be implemented in #22861

How did you do it?

Added a cancel_pr method that hits Elastictest endpoint API to cancel all previous job for the same PR

How did you verify/test it?

Verified locally and manually using CI

Any platform specific information?

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

Documentation

What is the motivation for this PR?
Currently, when running a new testplan, there is no safety check to cancel all the previous dangling job.

This add the ability to test_plan.py to cancel all related job with PR number using cancel_pr. However this PR does not add the call, only the implementation. The real call will be implemented in sonic-net#22861

How did you do it?
Added a cancel_pr method that hits Elastictest endpoint API to cancel all previous job for the same PR

How did you verify/test it?
Verified locally and manually using CI
Signed-off-by: Austin Pham <[email protected]>
Signed-off-by: mssonicbld <[email protected]>
@mssonicbld
Copy link
Collaborator Author

Original PR: #23101

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@auspham auspham closed this Mar 23, 2026
@auspham auspham reopened this Mar 23, 2026
@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit 6713ab2 into sonic-net:202511 Mar 23, 2026
19 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.

2 participants