Thanks for your pull request.

It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA) at https://cla.developers.google.com/.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check the information on your CLA or see this help article on setting the email on your git commits.

Once you've done that, please reply here to let us know. If you signed the CLA as a corporation, please let us know the company's name.

I don't understand why it hangs: if item is 0, then RegisterNodes is basically no-op. The pr lgtm, we don't have to wait.

I don't understand why it hangs: if item is 0, then RegisterNodes is basically no-op. The pr lgtm, we don't have to wait.

I made the change because it was stopping master start for two minutes. Without the short circuit for len(node.Items) == 0, it entered the outer for loop that would sleep until the retryCount was exceeded. In effect, having no nodes defined would cause a wait of about two minutes before the NodeController.Run would return. It caused long delays in master startup

ok, I see. I was thinking by "hangs", you mean k8s stop operating. I've tried my best to not delay cluster startup, but didn't recognize this case, good catch.