ggiamarchi / vagrant-openstack-provider

87%

master: 88%

LAST BUILD ON BRANCH fix-parallel-race-condition

branch: fix-parallel-race-condition

CHANGE BRANCH

x

Reset

• fix-parallel-race-condition
• add_sample_vagrantfiles
• attempt-func-tests
• auth_url_pattern
• bug#49
• bzk
• catch_ex
• doc#225
• doc-updates
• documentation-cosmetic
• error_handling
• extend-build-matrix
• fix-dependency-issue
• fix-floatingip
• fix-heat-stack-create
• fix-integration-tests
• fix-travis
• fix_matrix_ci_build
• fix_multiple_ips
• fix_permissions
• fix_ssh
• fix_travis
• fix_travis_build
• functionnal_tests
• heat
• issu146
• issue#105
• issue#106
• issue#112
• issue#128
• issue#132
• issue#132_test_travis_containers
• issue#142
• issue#160
• issue#168
• issue#185
• issue#186
• issue#193
• issue#194
• issue#195
• issue#205
• issue#24
• issue#248
• issue#44
• issue#49
• issue#51
• issue#52
• issue#54
• issue#87
• issue183
• issue218
• issue50
• issue55
• issue61
• issue68
• issue98
• master
• move-license
• nova_opts
• openstack_case
• quality#58
• release_0_7_0
• remove-sponsoring
• samples
• ssh-wip
• ssh_config
• support-http-proxy
• test-insert-key
• test_vars
• tests
• tmp_ggi
• update_readme
• upgrade_robocop
• v0.10.0
• v0.11.0
• v0.13.0
• v0.3.0
• v0.3.1
• v0.3.3
• v0.3.4.pre
• v0.3.x
• v0.4.0
• v0.4.1
• v0.4.x
• v0.5.x
• v0.6.0
• v0.6.1
• v0.7.0
• v0.7.1
• v0.7.2
• v0.8.0
• v0.9.0
• vagrant-1.8-matrix
• version-checker-update
• volume_boot_image
• website

Build # 821

Build Type

push

travis-ci

Committed by ggiamarchi

Commit Message

Fix deadlock for parallel `vagrant up`

When some floating IPs are allocated but unused and when
floating_ip_pool_always_allocate = false this can lead
into a situation where several servers are trying to use
the same IP address and it leads to a deadlock because
they all wait to have the IP assigned and of course only
one of them can have it.

The mutex has to be at a higher level than it was to
ensure floating IP assignment to be well done in an
atomic way.

Of course, doing that, floating IP assignment does not
occur in parallel anymore but it looks to be the only
way to solve the problem.

Run Details

1985 of 2279 relevant lines covered (87.1%)

44.37 hits per line