Skip to content
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

Closed
mml opened this issue Jun 23, 2016 · 8 comments
Closed

Federation: DNS resolution should work on Jenkins e2e tests #27949

mml opened this issue Jun 23, 2016 · 8 comments
Assignees
Labels
priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now.

Comments

@mml
Copy link
Contributor

mml commented Jun 23, 2016

this is just a not bot-noise-infested version of #27739

@mml mml added priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. area/cluster-federation labels Jun 23, 2016
@mml mml added this to the v1.3 milestone Jun 23, 2016
@mml mml self-assigned this Jun 23, 2016
@mml
Copy link
Contributor Author

mml commented Jun 23, 2016

@kubernetes/sig-cluster-federation

@mml
Copy link
Contributor Author

mml commented Jun 23, 2016

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.

I0623 16:22:47.630531       1 logs.go:41] skydns: incomplete CNAME chain from "federated-service.e2e-tests-service-jymu4.federation.svc.us-central1-f.kube.5yetis.net.": rcode 3 is not equal to success

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 ...svc.us-central1.kube... but not for ...svc.us-central1-f.kube.... Note -f.

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?

@quinton-hoole

@madhusudancs
Copy link
Contributor

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.

@mml
Copy link
Contributor Author

mml commented Jun 23, 2016

Awesome, thanks @madhusudancs.

@mml mml assigned madhusudancs and unassigned mml Jun 23, 2016
@madhusudancs
Copy link
Contributor

#27960 is the fix.

@ghost ghost changed the title ubernetes DNS resolution should work Federation: DNS resolution should work on Jenkins e2e tests Jun 23, 2016
@ghost ghost added the team/control-plane label Jun 23, 2016
@goltermann
Copy link
Contributor

@madhusudancs can this be closed? #27960 appears merged

@ghost
Copy link

ghost commented Jun 23, 2016

@goltermann Lets leave this open until we verify that DNS resolution e2e tests pass on Jenkins.

@goltermann goltermann modified the milestones: next-candidate, v1.3 Jun 24, 2016
@madhusudancs
Copy link
Contributor

This is fixed. Closing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now.
Projects
None yet
Development

No branches or pull requests

3 participants