-
Notifications
You must be signed in to change notification settings - Fork 4
Adding example for running tests in parallel in Azure #8
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
base: main
Are you sure you want to change the base?
Conversation
mw-hrastega
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @Vahila, I made some edits to the existing comments. Thanks.
| - task: RunMATLABBuild@1 | ||
| inputs: | ||
| tasks: equivalenceTest | ||
| env: | ||
| MLM_LICENSE_TOKEN: $(MLM_LICENSE_TOKEN) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aren't these tests being split and run again below?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The tests needs some artifacts which are generated by a task in buildfile. I updated the task to 'buildPythonPackage' which generates the artifact.
mw-hrastega
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for taking the time to update the comments, Vahila. I just noticed one small typo; otherwise, we should be all set.
| | Jenkins | [`Jenkins/CrossPlatformBuilder/Jenkinsfile`](https://github.com/mathworks/advanced-ci-configuration-examples/blob/main/Jenkins/CrossPlatformBuilder/Jenkinsfile) | | ||
|
|
||
| - **Run Tests across Multiple Agents**: Using parallel strategy in Azure DevOps run tests across multiple agents to speed up the testing process. [`AzureDevOps/ParallelStrategy.yml`](https://github.com/mathworks/advanced-ci-configuration-examples/blob/main/AzureDevOps/ParallelStrategy.yml) has an example of running tests on multiple agents. | ||
| - **Run Tests across Multiple Agents**: Using parallel strategy in Azure DevOps, run tests across multiple agents to speed up the testing process. [`AzureDevOps/ParallelStrategy.yml`](https://github.com/mathworks/advanced-ci-configuration-examples/blob/main/AzureDevOps/ParallelStrategy.yml) has an example of running tests on multiple agents. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggesting some edits:
- Run Tests Across Multiple Agents: Use the parallel strategy in Azure DevOps to run tests across multiple agents and speed up the testing process. For configuration details, see the example in
AzureDevOps/ParallelStrategy.yml.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The latter link is broken. Please make sure it gets fixed.
| For example, if there are multiple files [test1..test10] and 2 agents: | ||
| - Agent 1 runs tests from odd-numbered files. | ||
| - Agent 2 runs tests from even-numbered files. | ||
| For detailed slicing information, see https://docs.microsoft.com/en-us/vsts/pipelines/test/parallel-testing-any-test-runner |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't the link include learn. instead of docs? Here is what I am seeing in my browser:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's best to avoid redirects as much as possible.
| | GitHub Actions| [`.github/workflows/CrossPlatformBuilder.yml`](https://github.com/mathworks/advanced-ci-configuration-examples/blob/main/.github/workflows/CrossPlatformBuilder.yml) | | ||
| | Jenkins | [`Jenkins/CrossPlatformBuilder/Jenkinsfile`](https://github.com/mathworks/advanced-ci-configuration-examples/blob/main/Jenkins/CrossPlatformBuilder/Jenkinsfile) | | ||
|
|
||
| - **Run Tests across Multiple Agents**: Using parallel strategy in Azure DevOps, run tests across multiple agents to speed up the testing process. [`AzureDevOps/ParallelStrategy.yml`](https://github.com/mathworks/advanced-ci-configuration-examples/blob/main/AzureDevOps/ParallelStrategy.yml) has an example of running tests on multiple agents. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now that you are adding this paragraph, you also need to update line 17 to be inclusive of the new example. Something like this:
The repository contains examples for packaging and distributing a toolbox, building and uploading Python® packages, and running tests across multiple build agents.
Adding example for running tests in parallel in Azure