Stembuild specs: use Eventually() instead of Sleep

We now check for successful execution for 1 minute rather than sleeping
for 10 seconds and hoping that the VM is up and the guest-tools are
accepting connections.

Signed-off-by: Matthew Kocher <[email protected]>

Author: aramprice

stembuild/iaas_cli/iaas_clients/vcenter_govmomi_contract_test.go

@@ -49,10 +49,12 @@ var _ = Describe("VcenterClient", func() {
 			guestManager, err := vCenterManager.GuestManager(ctx, opsManager, envMustExist(TestVmUsername), envMustExist(TestVmPassword))
 			Expect(err).ToNot(HaveOccurred())
 
-			time.Sleep(10 * time.Second)
-
-			pid, err := guestManager.StartProgramInGuest(ctx, powershell, "Exit 59")
-			Expect(err).ToNot(HaveOccurred())
+			var pid int64
+			Eventually(func() error {
+				var startProgramErr error
+				pid, startProgramErr = guestManager.StartProgramInGuest(ctx, powershell, "Exit 59")
+				return startProgramErr
+			}).WithTimeout(1 * time.Minute).Should(Succeed())
 
 			exitCode, err := guestManager.ExitCodeForProgramInGuest(ctx, pid)
 			Expect(err).ToNot(HaveOccurred())