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

Remove outdated documented Mesos issue that service ports must be named #12616

Merged

Conversation

sttts
Copy link
Contributor

@sttts sttts commented Aug 12, 2015

The described behavior cannot be reproduced in current master anymore.

Compare d2iq-archive/kubernetes-mesos#322

The described behavior cannot be reproduced in current master anymore.

Compare d2iq-archive/kubernetes-mesos#322
@k8s-bot
Copy link

k8s-bot commented Aug 12, 2015

Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist")

If this message is too spammy, please complain to ixdy.

@sttts
Copy link
Contributor Author

sttts commented Aug 12, 2015

/cc @jdef

@cjcullen cjcullen self-assigned this Aug 12, 2015
cjcullen added a commit that referenced this pull request Aug 12, 2015
Remove outdated documented Mesos issue that service ports must be named
@cjcullen cjcullen merged commit 353d3b0 into kubernetes:master Aug 12, 2015
@cjcullen cjcullen added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 12, 2015
@jdef
Copy link
Contributor

jdef commented Aug 20, 2015

This needs to be reverted.

@sttts sttts deleted the mesos-service-port-name-issue branch September 14, 2015 09:14
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.

5 participants