-
Notifications
You must be signed in to change notification settings - Fork 38.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Federation][init-06] Check for the availability of federation API server's service loadbalancer address before waiting. #35863
[Federation][init-06] Check for the availability of federation API server's service loadbalancer address before waiting. #35863
Conversation
@@ -528,3 +528,13 @@ func printSuccess(cmdOut io.Writer, ips, hostnames []string) error { | |||
return err | |||
} | |||
|
|||
func pollImmediateInfinite(interval time.Duration, condition wait.ConditionFunc) error { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not put this in the wait library with PollInfinite?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One nit, then LGTM. Feel free to apply the label when done.
3e2236a
to
8c449cb
Compare
Jenkins GCI GKE smoke e2e failed for commit 8c449cbbfe2a7489c6e7e41c7841f13d50259191. Full PR test history. The magic incantation to run this job again is |
Jenkins GCE etcd3 e2e failed for commit 8c449cbbfe2a7489c6e7e41c7841f13d50259191. Full PR test history. The magic incantation to run this job again is |
Jenkins GCI GCE e2e failed for commit 8c449cbbfe2a7489c6e7e41c7841f13d50259191. Full PR test history. The magic incantation to run this job again is |
Jenkins unit/integration failed for commit 8c449cbbfe2a7489c6e7e41c7841f13d50259191. Full PR test history. The magic incantation to run this job again is |
Jenkins GCE e2e failed for commit 8c449cbbfe2a7489c6e7e41c7841f13d50259191. Full PR test history. The magic incantation to run this job again is |
Jenkins Kubemark GCE e2e failed for commit 8c449cbbfe2a7489c6e7e41c7841f13d50259191. Full PR test history. The magic incantation to run this job again is |
Jenkins GCE Node e2e failed for commit 8c449cbbfe2a7489c6e7e41c7841f13d50259191. Full PR test history. The magic incantation to run this job again is |
…rver's service loadbalancer address before waiting. This speeds up the tests. Otherwise tests end up unnecessarily waiting for the poll interval/duration which is 5 seconds right now.
8c449cb
to
caef02c
Compare
Jenkins verification failed for commit 8c449cbbfe2a7489c6e7e41c7841f13d50259191. Full PR test history. The magic incantation to run this job again is |
Jenkins GKE smoke e2e failed for commit caef02c. Full PR test history. The magic incantation to run this job again is |
@k8s-bot cvm gke e2e test this |
@quinton-hoole addressed the comment and rebased the PR. Adding the LGTM label. |
Please review only the last commit here. This is based on PR #35862 which will be reviewed independently.
Design Doc: PR #34484
cc @kubernetes/sig-cluster-federation @nikhiljindal
This change is![Reviewable](https://camo.githubusercontent.com/2d899f4291d07d3cd2fa4aaae1e3b243f164c23fce87d30a589ace0d496a444c/68747470733a2f2f72657669657761626c652e6b756265726e657465732e696f2f7265766965775f627574746f6e2e737667)