core: don't replace insecure key on base package [GH-5310]

hashicorp · Nov 19, 2015 · 1a7937e · 1a7937e · stepanzarubin · Dec 21, 2015
1 parent 42d0c36
commit 1a7937e
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/lib/vagrant/machine.rb b/lib/vagrant/machine.rb
@@ -116,6 +116,9 @@ def initialize(name, provider_name, provider_cls, provider_config, provider_opti
       # XXX: This is temporary. This will be removed very soon.
       if base
         @id = name
+
+        # For base setups, we don't want to insert the key
+        @config.ssh.insert_key = false
       else
         reload
       end

diff --git a/test/unit/vagrant/machine_test.rb b/test/unit/vagrant/machine_test.rb
@@ -70,6 +70,15 @@ def new_instance
       expect(subject.id).to be_nil
     end
 
+    describe "as a base" do
+      let(:base) { true}
+
+      it "should not insert key" do
+        subject = new_instance
+        expect(subject.config.ssh.insert_key).to be_false
+      end
+    end
+
     describe "communicator loading" do
       it "doesn't eager load SSH" do
         config.vm.communicator = :ssh