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
CI: K8sFQDNTest retry initial DNS lookups #10871
Conversation
test-focus K8sFQDNTest.* |
test-gke K8sFQDNTest.* |
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 small change request, lgtm otherwise
test/k8sT/fqdn.go
Outdated
worldInvalidTargetIP = addrs[0] | ||
|
||
return true | ||
}, "Could not get vagrant-cache.ci.cilium.io and jenkins.cilium.io IPs", &helpers.TimeoutConfig{Timeout: 30 * time.Second}) |
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.
Can we use helpers.MidCommandTimeout
?
bed68f5
to
7da61ac
Compare
test-gke K8sFQDNTest.* |
test-focus K8sFQDNTest.* |
build fails because of |
We target jenkins.cilium.io and vagrant-cache.ci.cilium.io as the remote targets for the FQDN tests. Sometimes, the DNS lookups fail and so does the test. We now retry for 30 seconds. When your DNS lookup at first doesn't succeed, try try again. Signed-off-by: Ray Bejjani <ray@isovalent.com>
7da61ac
to
307fb59
Compare
test-focus K8sFQDNTest.* |
test-gke K8sFQDNTest.* |
We target jenkins.cilium.io and vagrant-cache.ci.cilium.io as the remote
targets for the FQDN tests. Sometimes, the DNS lookups fail and so does
the test. We now retry for 30 seconds. When your DNS lookup at first
doesn't succeed, try try again.