ci: make external workload only use global DNS server in test #851
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Follow cilium/cilium#19483 Only use the first configured DNS server (the global one) in the test. Also add nslookup debug and retry options in case they are needed for further debugging and troubleshooting. This fixes external workload CI test on master that failed like this: Server: 169.254.169.254 Address: 169.254.169.254#53 ** server can't find clustermesh-apiserver.kube-system.svc.cluster.local: NXDOMAIN Here an incorrect nameserver is being used by nslookup. This 169.254.169.254 is configured (now) in GKE as a nameserver for some local names. Don't know why nslookup sometimes chooses it if no server address is given as a parameter. Co-authored-by: Jarno Rajahalme <[email protected]> Signed-off-by: Tobias Klauser <[email protected]>
…ing test Anoter follow-up for commit f3cf001 ("clustermesh: Run `cilium` with sudo in VM install script"). Signed-off-by: Tobias Klauser <[email protected]>
tklauser
force-pushed
the
pr/ci-externalworkloads-nslookup
branch
from
72c8abc to
8527ec9
Compare
|
Successful run of the external workloads test: https://github.com/cilium/cilium-cli/runs/6470646843?check_suite_focus=true Removing |
tklauser
force-pushed
the
pr/ci-externalworkloads-nslookup
branch
from
8527ec9 to
b0d1572
Compare
michi-covalent
approved these changes
May 17, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Follow cilium/cilium#19483
Only use the first configured DNS server (the global one) in the
test. Also add nslookup debug and retry options in case they are needed
for further debugging and troubleshooting.
This fixes external workload CI test on master that failed like this:
Here an incorrect nameserver is being used by nslookup. This
169.254.169.254 is configured (now) in GKE as a nameserver for some
local names. Don't know why nslookup sometimes chooses it if no server
address is given as a parameter.