-
Notifications
You must be signed in to change notification settings - Fork 467
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
Spec: Update the spec for federation #29
Comments
The federation records need to be totally revamped - last I looked at them they were ambiguous. @csbell @madhusudancs |
Welcome to the real guts of the project :)
From that doc: `mysvc.myns.myfed.svc.u7s.io` is proposed as an example.
This is ambiguous. It could just as likely mean a pod "mysvc" ins a
stateful set "myns" in namespace "myfed". The hope waas that the extra
token was enough to disambiguate but it's not. This snuck in without me
catching it, and now we need to find a way to fix it.
We proposed something like `mysvc.myns.myfed.fed.u7s.io` as an alternative,
but we need a plan to migrate people. Madhu might recall more details.
@bowei
…On Tue, Jan 17, 2017 at 6:53 AM, Christian Bell ***@***.***> wrote:
@thockin <https://github.com/thockin> The best reference outside of code
that I could find is located here
<kubernetes/kubernetes#25107 (comment)>.
By revamp, do you mean merging the current federation behavior into the
spec here or is there more involved?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#29 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AFVgVG4pgnZQtHDUi4LeMtLnLD_zlx8Rks5rTNX6gaJpZM4LjQ0A>
.
|
Given federation is still in its early stages, it would be great to get ahead of this issue |
Oh wow! Did not know about the existence of this repo and was hence missing the notifications.
This and in addition we had the literal There is also this doc that Quinton wrote - https://docs.google.com/document/d/1tSTgqSy7iR8cQx2z-bbqaYVITrraDQmV_aE43PuRPoI/edit#heading=h.ys8jzm2jzab9 |
Also ref kubernetes/kubernetes#27969
We had also discussed supporting DNS names for ingress. So
+1. |
@nikhiljindal One thing to keep in mind, increases in ndots can have a dramatic impact on performance. When a name has fewer dots than ndots, external names result in 10 DNS lookups/name. Modulo negative caching, this can easily result in DNS server meltdown. |
Issues go stale after 90d of inactivity. Prevent issues from auto-closing with an If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or |
Stale issues rot after 30d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or |
Rotten issues close after 30d of inactivity. Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
The current spec covers only a single cluster. It needs to be updated to reflect federation records and behavior.
The text was updated successfully, but these errors were encountered: