From fc76f71ffc7369eb41bd0d6fda0fe387e21fb572 Mon Sep 17 00:00:00 2001 From: Paul Hinze Date: Mon, 25 Jul 2022 15:52:11 -0500 Subject: [PATCH] Fix vbox multimachine by setting provider parallel default correctly Turns out I misinterpreted the behavior of a ruby method with a default being passed nil so I assumed :parallel was effectively defaulting to true when it is the opposite. --- plugins/commands/serve/service/internal_service.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/plugins/commands/serve/service/internal_service.rb b/plugins/commands/serve/service/internal_service.rb index 6325f671a..d1527dbd1 100644 --- a/plugins/commands/serve/service/internal_service.rb +++ b/plugins/commands/serve/service/internal_service.rb @@ -127,8 +127,8 @@ module VagrantPlugins return SDK::PluginInfo::ProviderOptions.new( # Priority is always set in V2::Plugin.provider priority: popts[:priority], - # Parallel is passed along to Environment#batch which defaults it to true - parallel: popts.fetch(:parallel, true), + # Parallel is defaults to falsy when its passed along as an arg to Environment#batch. + parallel: popts.fetch(:parallel, false), # BoxOptional defaults to falsy when it's used in Kernel_V2::VMConfig box_optional: popts.fetch(:box_optional, false), # Defaultable is considered true when it is not specified in Environment#default_provider