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

[202012] Split kvmtest t0 job into two jobs and run in parallel #10858

Merged
merged 4 commits into from
Jun 3, 2022

Conversation

saiarcot895
Copy link
Contributor

@saiarcot895 saiarcot895 commented May 17, 2022

Signed-off-by: Saikrishna Arcot sarcot@microsoft.com
(cherry picked from commit 4402872)

Why I did it

202012 branch of sonic-mgmt has the KVM T0 tests split up into 2 parts. However, the 202012 branch of sonic-buildimage doesn't know about these two parts, and so only half of the tests are being run.

How I did it

Cherry-pick the commit that added support for running these 2 parts from master branch to 202012 branch.

How to verify it

Verify that part 1 and part 2 both run, and that the time taken to run the tests is longer than the current time.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Why I did it
Introduce 2 sub jobs for kvmtest t0 job in sonic-mgmt repo in PR sonic-net/sonic-mgmt#4861
But in sonic-buildimage repo, because section parameter is null, it always run the part 2 test scripts in kvmtest t0 job.
It missed the part 1 test scripts in kvmtest.sh.

How I did it
Split kvmtest t0 job into two sub jobs such as sonic-mgmt repo and run them in parallel to save time.

How to verify it
Submit PR will trigger the pipeline to run.

Signed-off-by: Zhaohui Sun <zhaohuisun@microsoft.com>
(cherry picked from commit 4402872)
Copy link
Contributor

@ZhaohuiS ZhaohuiS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@saiarcot895 saiarcot895 marked this pull request as ready for review May 18, 2022 19:44
@saiarcot895
Copy link
Contributor Author

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@saiarcot895 saiarcot895 merged commit 3289598 into sonic-net:202012 Jun 3, 2022
@saiarcot895 saiarcot895 deleted the 202012-fix-test-runs branch June 3, 2022 16:39
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.

None yet

3 participants