Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Allow dots in resource names #1197

Merged
merged 1 commit into from
Jul 4, 2018
Merged

Conversation

squaremo
Copy link
Member

@squaremo squaremo commented Jul 3, 2018

Kubernetes resource names are DNS_SUBDOMAINs, which are basically
DNS_LABELs joined together with dots. It's perfectly reasonable to
have a name foo.bar.baz, for example.

Fixes #1195.

Kubernetes resource names are DNS_SUBDOMAINs, which are basically
DNS_LABELs joined together with dots. It's perfectly reasonable to
have a name `foo.bar.baz`, for example.
@squaremo squaremo requested a review from aaron7 July 3, 2018 15:47
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@squaremo squaremo removed the request for review from aaron7 July 4, 2018 07:18
@squaremo squaremo merged commit d2c48cb into master Jul 4, 2018
@squaremo squaremo deleted the issue/1195-resources-names-have-dots branch July 4, 2018 07:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants