kubectlWait function can loop forever

[bboreham]

This loop:

wksctl/pkg/plan/resource/kubectl_wait.go

Lines 65 to 73 in 2e2da9c

	for {
	cmd := fmt.Sprintf("kubectl get %q %s%s", args.WaitType, waitOn(args), waitNamespace(args))
	output, err := r.RunCommand(withoutProxy(cmd), nil)
	if err != nil || strings.Contains(output, "No resources found") {
	time.Sleep(500 * time.Millisecond)
	} else {
	break
	}
	}

will go forever if RunCommand() keeps erroring.

I think at least it should stop after the WaitTimeout, and probably return immediately on unexpected error.