vagrant-skytap 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ee9cca3ef03d038cdb2470d8311a73a4d8b2abd6
-  data.tar.gz: 8d8a3248ef15261b24731511665da4ae73ea90b4
+  metadata.gz: 42367c7e3c19aaeea00c74ed4756d8c54d92612f
+  data.tar.gz: c5b4962d7052fad5e4a7e4c90bad7799c64b92ed
 SHA512:
-  metadata.gz: e8cfb11207527c5d94ce6b50b1c5735b24f2080b6565d6977173bdfa9317072b6173b2b42f672fbfe080f2fd2d331b68e59de0b523c82085253fb2b752a967fb
-  data.tar.gz: 88af9755df93f2e2a1514bc86b1c3600561aa25e0dff8adf2695563b7ecf23c82463d03a976783aca0a09e7c3565c5d7209913194d28710ea7ec3ada34307ce9
+  metadata.gz: 3ac68aee81962e0729980aa6b5785413ca24780a2a149447f34ca3aec45a4cd4629c415035ff11c3acb838c69a85e5c115c47df2ea7ac24ded5655d0de8ed70c
+  data.tar.gz: a0fa8b2edc3ac422141d5af6d07990e18824c76443c649b2272ff0dab506124679b7fa401531e6ee215e38b5d14250a347b19d28c2c538e98ed3150190763799

data/.byebug_history

@@ -1,4 +1,12 @@
 c
+q
+action_result[:api_client].url
+action_result[:api_client].base_url
+action_result[:api_client]
+action_result.keys
+action_result.class
+action_result
+c
 $!
 c
 @env[:initial_states]
@@ -246,11 +254,3 @@ b 96
 b
 names
 finish
-$!
-n
-c
-n
-folders.count
-c
-folders.first
-folders.count

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+# 0.2.2 (January 26, 2016)
+
+* Bug fix: `vagrant add` would create but not start a new VM if
+  the first existing VM was already running.
+
 # 0.2.1 (January 26, 2016)
 
 * Handle error when the user tries to use --install-provider flag.

data/lib/vagrant-skytap/action/run_environment.rb

@@ -3,12 +3,9 @@ require "log4r"
 module VagrantPlugins
   module Skytap
     module Action
-      # Runs multiple VMs in parallel. This middleware will be invoked for
-      # each machine, so we set the :first_machine flag to prevent redundant
-      # REST calls.
+      # Runs multiple VMs in parallel. Ensures that the REST call happens
+      # only once.
       class RunEnvironment
-        include Vagrant::Util::Retryable
-
         attr_reader :env
 
         def initialize(app, env)
@@ -18,18 +15,20 @@ module VagrantPlugins
         end
 
         def call(env)
-          if env[:first_machine]
+          environment = env[:environment]
+          vm_ids = vm_ids_to_run
+
+          if environment.current_vm.id == vm_ids.first
             env[:ui].info(I18n.t("vagrant_skytap.running_environment"))
-            vm_ids = env[:machines].try(:collect, &:id)
             @logger.info("Running VMs: #{vm_ids}")
-            env[:environment].run!(vm_ids)
+            environment.run!(vm_ids)
           end
 
           @app.call(env)
         end
 
-        def environment
-          env[:environment]
+        def vm_ids_to_run
+          @env[:initial_states].reject{|id, state| state == :running}.keys
         end
       end
     end

data/lib/vagrant-skytap/command/up.rb

@@ -94,9 +94,8 @@ module VagrantPlugins
 
             initial_states = machines.inject({}) {|acc, m| acc[m.id] = m.state.id ; acc }
 
-            machines.each_with_index do |machine, i|
+            machines.each do |machine|
               machine.action(:run_vm, options.merge(
-                first_machine:  i == 0,
                 machines:       machines,
                 initial_states: initial_states
               ))

data/lib/vagrant-skytap/version.rb

@@ -1,5 +1,5 @@
 module VagrantPlugins
   module Skytap
-    VERSION = "0.2.1"
+    VERSION = "0.2.2"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: vagrant-skytap
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - Nick Astete