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

Rename type Port to ContainerPort #4916

Merged
merged 1 commit into from
Mar 2, 2015

Conversation

thockin
Copy link
Member

@thockin thockin commented Feb 27, 2015

Clearing way for multi-port services. Sadly I had to do this by hand - I just could not get gorename to fix up
users of it.

Sadly I had to do this by hand - I just could not get gorename to fix up
users of it.
@thockin thockin force-pushed the plural_21_endpoints branch from 3af8f9b to ea548b8 Compare February 28, 2015 05:06
@bgrant0607 bgrant0607 added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Mar 2, 2015
@bgrant0607
Copy link
Member

LGTM

@bgrant0607 bgrant0607 self-assigned this Mar 2, 2015
brendandburns added a commit that referenced this pull request Mar 2, 2015
Rename type Port to ContainerPort
@brendandburns brendandburns merged commit fe97346 into kubernetes:master Mar 2, 2015
@thockin thockin deleted the plural_21_endpoints branch March 7, 2015 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants