From 83495a4f8b23e779cb67e7d99ab94c8c62bcaea4 Mon Sep 17 00:00:00 2001
From: sophia <scastellarin95@gmail.com>
Date: Tue, 10 Aug 2021 14:41:42 -0400
Subject: [PATCH] Fix each machine index machine yielder

---
 internal/core/target_index.go       |  1 -
 lib/vagrant/machine_index/remote.rb | 17 ++++++++++++-----
 2 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/internal/core/target_index.go b/internal/core/target_index.go
index 3861c3563..7b977f618 100644
--- a/internal/core/target_index.go
+++ b/internal/core/target_index.go
@@ -68,7 +68,6 @@ func (t *TargetIndex) Set(entry core.Target) (updatedEntry core.Target, err erro
 			Target: target.target,
 		},
 	)
-	// TODO: check if this actually gets back a full target
 	updatedEntry, err = NewTarget(
 		t.ctx,
 		WithTargetName(updatedTarget.Target.Name),
diff --git a/lib/vagrant/machine_index/remote.rb b/lib/vagrant/machine_index/remote.rb
index 70b400a3c..6b4abfbaa 100644
--- a/lib/vagrant/machine_index/remote.rb
+++ b/lib/vagrant/machine_index/remote.rb
@@ -76,16 +76,21 @@ module Vagrant
       end
 
       # Iterate over every machine in the index
-      def each(reload=false)
+      def each(reload=true)
         if reload
           arg_machines = @client.all()
+          @logger.debug("machines args: #{arg_machines}")
+
           arg_machines.each do |m|
-            @machines << machine_to_entry(m)
+            e = machine_to_entry(m)
+            @machines[e.id] = e
           end
         end
 
+        @logger.debug("machines: #{@machines.keys}")
+
         @machines.each do |uuid, data|
-          yield Entry.new(uuid, data.merge("id" => uuid))
+          yield data
         end
       end
 
@@ -108,13 +113,15 @@ module Vagrant
           "local_data_path" => machine_client.get_local_data_path(),
           # TODO: get the provider!
           "provider" => "virtualbox",
-          "state" => machine_client.get_state(),
+          "state" => machine_client.get_state().id,
           "vagrantfile_name" => machine_client.get_vagrantfile_name(),
           "vagrantfile_path" => machine_client.get_vagrantfile_path(),
           "machine_client" => machine_client,
         }
+        id = machine_client.get_id()
+        @logger.debug("machine id: #{id}")
         entry = Vagrant::MachineIndex::Entry.new(
-          id=machine_client.resource_id, raw=raw
+          id=id, raw=raw
         )
         return entry
       end