Issue #192: HostSystem doesn't seem to be returning the correct host.

compute_resource.go

@@ -16,12 +16,35 @@ limitations under the License.
 
 package govmomi
 
-import "github.com/vmware/govmomi/vim25/types"
+import (
+	"github.com/vmware/govmomi/vim25/mo"
+	"github.com/vmware/govmomi/vim25/types"
+)
 
 type ComputeResource struct {
 	types.ManagedObjectReference
+
+	c *Client
+}
+
+func NewComputeResource(c *Client, ref types.ManagedObjectReference) *ComputeResource {
+	return &ComputeResource{
+		ManagedObjectReference: ref,
+		c: c,
+	}
 }
 
 func (c ComputeResource) Reference() types.ManagedObjectReference {
 	return c.ManagedObjectReference
 }
+
+func (c ComputeResource) Hosts() ([]types.ManagedObjectReference, error) {
+	var cr mo.ComputeResource
+	ps := []string{"host"}
+
+	err := c.c.Properties(c.Reference(), ps, &cr)
+	if err != nil {
+		return nil, err
+	}
+	return cr.Host, nil
+}

find/finder.go

@@ -312,11 +312,23 @@ func (f *Finder) HostSystemList(path ...string) ([]*govmomi.HostSystem, error) {
 
 	var hss []*govmomi.HostSystem
 	for _, e := range es {
+		var hs *govmomi.HostSystem
+
 		switch o := e.Object.(type) {
 		case mo.HostSystem:
-			hs := govmomi.NewHostSystem(f.Client, o.Reference())
-			hss = append(hss, hs)
+			hs = govmomi.NewHostSystem(f.Client, o.Reference())
+		case mo.ComputeResource:
+			cr := govmomi.NewComputeResource(f.Client, o.Reference())
+			hosts, err := cr.Hosts()
+			if err != nil {
+				return nil, err
+			}
+			hs = govmomi.NewHostSystem(f.Client, hosts[0])
+		default:
+			continue
 		}
+
+		hss = append(hss, hs)
 	}
 
 	return hss, nil

govc/test/host.bats

@@ -23,6 +23,10 @@ load test_helper
   assert_success
   grep -q Manufacturer: <<<$output
 
+  run govc host.info -host ${name##*/}
+  assert_success
+  grep -q Manufacturer: <<<$output
+
   run govc host.info -host.ipath $name
   assert_success