diff options
| author | Christopher Baines <mail@cbaines.net> | 2018-07-04 12:06:25 +0100 |
|---|---|---|
| committer | Christopher Baines <mail@cbaines.net> | 2018-07-04 12:06:25 +0100 |
| commit | 8bd3eac92e0b098608b4b4f0becc9516d872213a (patch) | |
| tree | c6f087725bef61be23079d8a8582fad5462eaf14 /app/models/backends/terraform_libvirt/mini_environment_methods.rb | |
| parent | 74fb581d6aedc321b64e43c82bff01bc35a0f7da (diff) | |
| download | govuk-mini-environment-admin-8bd3eac92e0b098608b4b4f0becc9516d872213a.tar govuk-mini-environment-admin-8bd3eac92e0b098608b4b4f0becc9516d872213a.tar.gz | |
Use database locking to prevent concurrent Terraform jobs
Currently, the Terraform code uses chdir in the Ruby process to change
to the right working directory. This affects all threads in the
process, and thus means that all Terraform jobs conflict with each
other.
It would be good to fix this somehow, e.g. not changing directory, and
just specifying the directory in which to run Terraform when starting
it, but for now, use the advisory locks to force the jobs to run
sequentially.
Diffstat (limited to 'app/models/backends/terraform_libvirt/mini_environment_methods.rb')
| -rw-r--r-- | app/models/backends/terraform_libvirt/mini_environment_methods.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/models/backends/terraform_libvirt/mini_environment_methods.rb b/app/models/backends/terraform_libvirt/mini_environment_methods.rb index 9b68d49..ef2b4c1 100644 --- a/app/models/backends/terraform_libvirt/mini_environment_methods.rb +++ b/app/models/backends/terraform_libvirt/mini_environment_methods.rb @@ -79,7 +79,7 @@ module Backends::TerraformLibvirt::MiniEnvironmentMethods def within_terraform_working_directory(mini_environment, &block) with_advisory_lock( - "libvirt_mini_environment_terraform_working_directory/#{mini_environment.id}" + "terraform" ) do TerraformWorkingDirectory.new( mini_environment_state_id(mini_environment), |