explicitly reserve kube- prefix for system namespaces#2379
explicitly reserve kube- prefix for system namespaces#2379k8s-ci-robot merged 1 commit intokubernetes:masterfrom
Conversation
| called `Fooable`, not `IsFooable`. | ||
|
|
||
| ### Namespace Names | ||
| * The name of a namespace must be DNS compatible. |
There was a problem hiding this comment.
c919f78 to
4a8ea25
Compare
|
Thanks. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bgrant0607 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
I think this is necessary, and has previously been done by convention piece-mill. Good to see it codified. This is worth communicating to dev/user lists, since it can affect what namespaces people choose for their own projects, and how cluster deployers choose to shape custom policies. |
|
If we want a prefix reserved for providers, I'd be open to do that. Perhaps something like |
|
This is a weird doc to document this in... |
|
@thockin do you have a better suggestion? |
|
It's not part of the API it's more like runtime conventions. That said, we
don't have many (any?) others, so there's no doc to cover it.
…On Fri, Jul 27, 2018, 5:27 PM Michael Taufen ***@***.***> wrote:
@thockin <https://github.com/thockin> do you have a better suggestion?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2379 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFVgVOw6K67bStd-qhavgYl7f6hzmGJBks5uK6_vgaJpZM4VTowB>
.
|
Based on an offline discussion with @bgrant0607, this is the existing convention and we should stick to it for any new system namespaces, and we can't use the
*.k8s.iopattern, because additional dots can mess up the format for service DNS entries.Users should absolutely not be using the
kube-prefix for non-system namespaces./cc @bgrant0607 @liggitt