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

GRPC header based consistent hashing #12803

Open
mistborn17 opened this issue Feb 7, 2025 · 6 comments
Open

GRPC header based consistent hashing #12803

mistborn17 opened this issue Feb 7, 2025 · 6 comments
Labels
kind/feature Categorizes issue or PR as related to a new feature. needs-priority triage/needs-information Indicates an issue needs more information in order to work on it.

Comments

@mistborn17
Copy link

Is there support for gRPC header token based consistent hashing ?

@mistborn17 mistborn17 added the kind/feature Categorizes issue or PR as related to a new feature. label Feb 7, 2025
@k8s-ci-robot k8s-ci-robot added needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. needs-priority labels Feb 7, 2025
@longwuyuan
Copy link
Contributor

Not heard of it. Is the hashing used for sticky sessions or something else ?

@mistborn17
Copy link
Author

If the gRPC server is stateful, then the given sequence of RPCs needs to land on the same kubernetes pod.

For ex, RPC1-upload document. RPC2 - index the document, RPC3 - track indexing status, should land on the same pod where the document is uploaded.

@longwuyuan
Copy link
Contributor

longwuyuan commented Feb 7, 2025 via email

@strongjz
Copy link
Member

@mistborn17 Can you test if those upstream hashes will also work for your GRPC service? If not, I would say no, and we would potentially not support them as we are working towards getting Ingate up and running. PRs are always welcome, with appropriate e2e tests.

@strongjz
Copy link
Member

/triage needs-information

@k8s-ci-robot k8s-ci-robot added triage/needs-information Indicates an issue needs more information in order to work on it. and removed needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Feb 13, 2025
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. needs-priority triage/needs-information Indicates an issue needs more information in order to work on it.
Projects
Development

No branches or pull requests

4 participants