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

Explicit parallelization support #126

Merged
merged 1 commit into from
Jul 9, 2015
Merged

Explicit parallelization support #126

merged 1 commit into from
Jul 9, 2015

Conversation

xlucas
Copy link
Contributor

@xlucas xlucas commented Mar 27, 2015

This declares the provider as capable of using parallelization by default. Without this, VMs are created sequentially and it can take a long time, for instance when extensively deploying from templates (see #32). In this case this is a huge time saver.

It can still be disabled with the --no-parallel option.

@xlucas xlucas changed the title Explicit parallelization support resolves #32 Explicit parallelization support Mar 27, 2015
@rylarson
Copy link
Contributor

rylarson commented May 7, 2015

Have you tested this fully? I did this a while back and I had problems bringing up many VM's in parallel. I spent some time chasing it down and I ran out of time to fix all of the locking required to make it work reliably.

@arhimondr
Copy link

I've tested it. Looks like it works well. Which kind of problems you had?

@rylarson
Copy link
Contributor

It is possible that some unrelated changes in the plugin since I tried it out have fixed it since I tried it. I will give it another shot this week in my environment. We are running vSphere 5.5 and when I tried to bring up 100 VM's at once with the parallel provider, it would fail about 2-3 of them pretty consistently due to weird exceptions that seemed to be related to some rbvmomi objects not being thread safe.

@arhimondr
Copy link

It was looking stable for me because i have cluster with only 3 machines.
But recently i got similar error.

michaeljb added a commit that referenced this pull request Jul 9, 2015
@michaeljb michaeljb merged commit 4a842b9 into nsidc:master Jul 9, 2015
michaeljb added a commit that referenced this pull request Jul 9, 2015
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

4 participants