Skip to content

Add INCLUDE_JOBS parameter to pre_defined_pr_test pipeline#23244

Merged
xwjiang-ms merged 1 commit intosonic-net:masterfrom
xwjiang-ms:modify_pre_defined_pr_test
Mar 24, 2026
Merged

Add INCLUDE_JOBS parameter to pre_defined_pr_test pipeline#23244
xwjiang-ms merged 1 commit intosonic-net:masterfrom
xwjiang-ms:modify_pre_defined_pr_test

Conversation

@xwjiang-ms
Copy link
Copy Markdown
Contributor

@xwjiang-ms xwjiang-ms commented Mar 24, 2026

Description of PR

Summary:
Pass the INCLUDE_JOBS parameter through pre_defined_pr_test.yml so that pre-defined PR test runs can execute the full job set (default "all"), complementing the change introduced in #23123.

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?

#23123 added INCLUDE_JOBS to pr_test_template.yml with a default of t0_job,t1_job,t2_job so that regular PR tests stay scoped to t0/t1/t2, while baseline tests pass INCLUDE_JOBS: all to run every job. However, pre_defined_pr_test.yml was not updated and therefore could not pass INCLUDE_JOBS through to the shared template. Pre-defined PR tests should be able to run all tests, so this parameter needs to be plumbed through.

How did you do it?

  • Added a new INCLUDE_JOBS parameter (type: string, default: "all") to pre_defined_pr_test.yml.
  • Passed INCLUDE_JOBS into the stage template variables so the downstream pr_test_template.yml receives it and enables all jobs by default.

How did you verify/test it?

Verified the pipeline YAML syntax is valid and the parameter is correctly forwarded to the stage template variables.

Any platform specific information?

N/A

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

N/A

Documentation

N/A

Signed-off-by: xiaweijiang <xiaweijiang@microsoft.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@xwjiang-ms xwjiang-ms changed the title Update pre_defined_pr_test yaml Add INCLUDE_JOBS parameter to pre_defined_pr_test pipeline Mar 24, 2026
@xwjiang-ms xwjiang-ms merged commit abd85d6 into sonic-net:master Mar 24, 2026
18 checks passed
@xwjiang-ms xwjiang-ms deleted the modify_pre_defined_pr_test branch March 24, 2026 23:33
ravaliyel pushed a commit to ravaliyel/sonic-mgmt that referenced this pull request Mar 27, 2026
Signed-off-by: xiaweijiang <xiaweijiang@microsoft.com>
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.

3 participants