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

Add load balancing support to services. #135

Merged
merged 9 commits into from
Jun 18, 2014
Merged

Conversation

brendandburns
Copy link
Contributor

No description provided.

@lavalamp
Copy link
Member

Hm, I see some binary files and .hg directories from dependencies. Is it a problem for the next person to re-get if we add them to .gitignore?

@brendandburns
Copy link
Contributor Author

Ok, addressed comments, removed the .hg files, cleaned things up. I think its all set now.

PTAL
Thanks!
--brendan

@brendandburns
Copy link
Contributor Author

(note I still need to add unit tests, so don't merge. Once there's LGTM, I'll add tests.)

@lavalamp
Copy link
Member

LGTM now. One nit, cloudprovider.CloudInterface could be named just cloudprovider.Interface.

@brendandburns
Copy link
Contributor Author

Comments addressed. Tests added. Ready to merge when you're ready.

lavalamp added a commit that referenced this pull request Jun 18, 2014
Add load balancing support to services.
@lavalamp lavalamp merged commit 246db91 into kubernetes:master Jun 18, 2014
vishh pushed a commit to vishh/kubernetes that referenced this pull request Apr 6, 2016
iaguis pushed a commit to kinvolk/kubernetes that referenced this pull request Feb 6, 2018
Add CAP_NET_BIND_SERVICE to hyperkube.
marun added a commit to marun/kubernetes that referenced this pull request Jun 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants