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

Specify load-balancing behavior for each service that is being proxied #98

Closed
hbagdi opened this issue Feb 15, 2020 · 9 comments
Closed
Labels
kind/feature Categorizes issue or PR as related to a new feature. kind/user-story Categorizes an issue as capturing a user story lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness.

Comments

@hbagdi
Copy link
Contributor

hbagdi commented Feb 15, 2020

What would you like to be added:

For any service that is exposed via service-apis, a service owner should be able to specify the load-balancing behavior that should be used. This can include a common set of algorithms like round-robin, weighted, hash based routing, cookie/session based routing.

Why is this needed:

Most services in Kubernetes are designed to scale horizontally. A lot of legacy services that are ported to run on Kubernetes have the requirement of stick sessions. Most proxies and load balancers support load-balancing as well and this is a widely adopted feature in most of current ingress implementations.

@hbagdi hbagdi added the kind/feature Categorizes issue or PR as related to a new feature. label Feb 15, 2020
@hbagdi
Copy link
Contributor Author

hbagdi commented Feb 15, 2020

There is certainly an overlap with kube-proxy and the corev1.Service resource here.

@bowei
Copy link
Contributor

bowei commented Feb 15, 2020

Yes, this API would give us an opportunity to add better load balancing knobs without adding more to the core.Service bucket

@hbagdi
Copy link
Contributor Author

hbagdi commented Mar 20, 2020

@robscott Seems like there is already an issue open for load-balancing.

@hbagdi
Copy link
Contributor Author

hbagdi commented Mar 20, 2020

/kind user-story

@k8s-ci-robot k8s-ci-robot added the kind/user-story Categorizes an issue as capturing a user story label Mar 20, 2020
@fejta-bot
Copy link

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jun 18, 2020
@bowei bowei removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jul 9, 2020
@fejta-bot
Copy link

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Oct 8, 2020
@jpeach
Copy link
Contributor

jpeach commented Oct 8, 2020

/remove-lifecycle stale
/lifecycle frozen

@k8s-ci-robot k8s-ci-robot added lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Oct 8, 2020
@hbagdi
Copy link
Contributor Author

hbagdi commented Aug 24, 2021

In search for a solution, we were unsuccessful in coming up with a portable way to support this feature.
As a compromise, we came up with an easy and portable way to support such implementation-specific features in a consistent manner. The solution is documented in #713.

If there are more portable ways to tackle this, we would love to hear about them.

For now, I'll close this issue.
/close

@k8s-ci-robot
Copy link
Contributor

@hbagdi: Closing this issue.

In response to this:

In search for a solution, we were unsuccessful in coming up with a portable way to support this feature.
As a compromise, we came up with an easy and portable way to support such implementation-specific features in a consistent manner. The solution is documented in #713.

If there are more portable ways to tackle this, we would love to hear about them.

For now, I'll close this issue.
/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature. kind/user-story Categorizes an issue as capturing a user story lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness.
Projects
None yet
Development

No branches or pull requests

5 participants