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

Run module tests in parallel #1262

Merged
merged 2 commits into from
Nov 8, 2016
Merged

Run module tests in parallel #1262

merged 2 commits into from
Nov 8, 2016

Conversation

troydai
Copy link
Contributor

@troydai troydai commented Nov 8, 2016

Running test modules in parallel. On my machine it reduces running time from 16 minutes to around 9 minutes.

The actual running time is shorter then that. I'm going to profile the test further to figure out where were the time spent on.

Summary
========================
acr        skipped                     
acs        passed  16:38:04  16:38:07  
cloud      skipped                     
component  skipped                     
configure  skipped                     
container  skipped                     
context    skipped                     
feedback   skipped                     
iot        passed  16:38:04  16:38:13  
keyvault   passed  16:38:04  16:38:31  
network    passed  16:38:04  16:42:15  
profile    skipped                     
redis      passed  16:38:04  16:38:07  
resource   passed  16:38:04  16:39:23  
role       passed  16:38:04  16:38:46  
storage    passed  16:38:04  16:39:43  
taskhelp   skipped                     
vm         passed  16:38:04  16:42:45  
webapp     passed  16:38:04  16:39:28  
========================

Running time on CI down to 29 minutes:
screen shot 2016-11-08 at 9 49 20 am

And the summary on the CI looks like:
screen shot 2016-11-08 at 9 55 08 am

Copy link
Member

@tjprescott tjprescott left a comment

Choose a reason for hiding this comment

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

LGTM. It think it would be nice to output the elapsed time for each series in addition to (or even in lieu of) the start and stop times.

Recommend @derekbekoe take a look as well since he originally wrote this.

@troydai
Copy link
Contributor Author

troydai commented Nov 8, 2016

Updated.

  1. Print duration instead of finished time;
  2. Correctly print the output if one test module test fails.

@troydai troydai merged commit 7495dc6 into Azure:master Nov 8, 2016
@troydai troydai deleted the trdai-parallel-test branch November 8, 2016 18:27
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.

5 participants