Fix regression where net status not added

akutz · akutz · commit f4d82affc497 · 2019-06-25T18:52:53.000-05:00
This patch fixes a regression where the network status was not added to
the machine, and therefore the machine was never in the ready state.
diff --git a/pkg/cloud/vsphere/services/govmomi/update.go b/pkg/cloud/vsphere/services/govmomi/update.go
@@ -107,6 +107,7 @@ func reconcileNetwork(ctx *context.MachineContext, vm *object.VirtualMachine) er
 	if expNetCount != actNetCount {
 		return errors.Errorf("invalid network count for %q: exp=%d act=%d", ctx, expNetCount, actNetCount)
 	}
+	ctx.MachineStatus.Network = allNetStatus
 
 	// Update the MAC addresses in the machine's network config as well. This
 	// is required in order to generate the metadata.

Original file line number	Diff line number	Diff line change
`@@ -107,6 +107,7 @@ func reconcileNetwork(ctx context.MachineContext, vm object.VirtualMachine) er`
`107`	`107`	`if expNetCount != actNetCount {`
`108`	`108`	`return errors.Errorf("invalid network count for %q: exp=%d act=%d", ctx, expNetCount, actNetCount)`
`109`	`109`	`}`
	`110`	`+ ctx.MachineStatus.Network = allNetStatus`
`110`	`111`
`111`	`112`	`// Update the MAC addresses in the machine's network config as well. This`
`112`	`113`	`// is required in order to generate the metadata.`