-
Notifications
You must be signed in to change notification settings - Fork 40k
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: DNS resolution should work on Jenkins e2e tests #27949
Comments
@kubernetes/sig-cluster-federation |
I think I see a problem. The names all look very similar, and so it's easy to glance at a debug log and think you know what's going on.
I thought that name looked like it should work, but here are the actual names in the record set % gcloud dns record-sets list -z kube | grep jymu4
federated-service.e2e-tests-service-jymu4.federation.svc.kube.5yetis.net. A 180 130.211.115.153
federated-service.e2e-tests-service-jymu4.federation.svc.us-central1.kube.5yetis.net. A 180 130.211.115.153
federated-service.e2e-tests-service-jymu4.federation.svc.us-central1-f.us-central1.kube.5yetis.net. A 180 130.211.115.153 Note that we have a record for So, I assume the records in cloud DNS are correct and the zone name kube-dns is appending is wrong, and we shoud be using us-central1 instead. Correct? |
I will send a bug fix to this. I think this was due to a miscommunication between @quinton-hoole and myself about the way each of us were generating the names. I should have caught this in the code review, sorry. |
Awesome, thanks @madhusudancs. |
#27960 is the fix. |
@madhusudancs can this be closed? #27960 appears merged |
@goltermann Lets leave this open until we verify that DNS resolution e2e tests pass on Jenkins. |
This is fixed. Closing it. |
this is just a not bot-noise-infested version of #27739
The text was updated successfully, but these errors were encountered: