commands/up: flag --no-destroy-on-error [GH-2011]

CHANGELOG.md

@@ -11,6 +11,8 @@ FEATURES:
     `vagrant destroy`. [GH-1302]
   - Chef Client provisioner will now clean up the node/client using
     `knife` if configured to do so.
+  - `vagrant up` has a `--no-destroy-on-error` flag that will not destroy
+    the VM if a fatal error occurs. [GH-2011]
 
 IMPROVEMENTS:
 

plugins/commands/up/command.rb

@@ -11,6 +11,7 @@ module VagrantPlugins
 
       def execute
         options = {}
+        options[:destroy_on_error] = true
         options[:parallel] = true
 
         opts = OptionParser.new do |o|
@@ -19,6 +20,11 @@ module VagrantPlugins
 
           build_start_options(o, options)
 
+          o.on("--[no-]destroy-on-error",
+               "Destroy machine if any fatal error happens (default to true).") do |destroy|
+            options[:destroy_on_error] = destroy
+          end
+
           o.on("--[no-]parallel",
                "Enable or disable parallelism if provider supports it.") do |parallel|
             options[:parallel] = parallel

plugins/providers/virtualbox/action/import.rb

@@ -36,6 +36,9 @@ module VagrantPlugins
           if env[:machine].provider.state.id != :not_created
             return if env["vagrant.error"].is_a?(Vagrant::Errors::VagrantError)
 
+            # If we're not supposed to destroy on error then just return
+            return if !env[:destroy_on_error]
+
             # Interrupted, destroy the VM. We note that we don't want to
             # validate the configuration here, and we don't want to confirm
             # we want to destroy.