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

Added configurable parallelism pool size #1177

Merged
merged 3 commits into from
Oct 27, 2020

Conversation

dmattia
Copy link
Contributor

@dmattia dmattia commented Sep 3, 2020

No description provided.

@dmattia dmattia force-pushed the dmattia/concurrency_flag branch from e19b845 to d1fe029 Compare September 3, 2020 15:51
@codecov
Copy link

codecov bot commented Sep 3, 2020

Codecov Report

Merging #1177 into master will increase coverage by 0.01%.
The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1177      +/-   ##
==========================================
+ Coverage   70.71%   70.72%   +0.01%     
==========================================
  Files          71       71              
  Lines        5975     5978       +3     
==========================================
+ Hits         4225     4228       +3     
  Misses       1398     1398              
  Partials      352      352              
Impacted Files Coverage Δ
server/events/command_runner.go 53.92% <0.00%> (ø)
server/user_config.go 100.00% <ø> (ø)
cmd/server.go 79.75% <100.00%> (+0.16%) ⬆️
server/server.go 60.80% <100.00%> (+0.09%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 489b41c...c24c500. Read the comment docs.

@dmattia dmattia marked this pull request as ready for review September 3, 2020 16:13
@dmattia
Copy link
Contributor Author

dmattia commented Oct 26, 2020

On a PR with 412 plans being run in parallel, it takes 37 minutes without this change, and 6 minutes when I increase the parallelism limit on a repo of mine.

@antonbabenko I think this is ready for a review if you ever have a moment 😄

@lkysow lkysow merged commit c789e1a into runatlantis:master Oct 27, 2020
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.

2 participants