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

docs(ingress): use GRPC instead of HTTP2 for AWS #21029

Merged
merged 1 commit into from
Dec 3, 2024

Conversation

agaudreault
Copy link
Member

@agaudreault agaudreault commented Dec 3, 2024

Clarify documentation on GRPC ingress with ALB. When HTTP2 is used, the target group will be in a unhealthy state because the HTTP2 health check is downgraded to HTTP1 by argo (Argo-server does not actually support http2, it downgrades everything non-grpc to HTTP1).

When using HTTP2 with Pod Readiness Gates, the pods will never show as "Ready" because the associated target group with the grpc service will never be healthy (due to the invalid health check).

This also matches the default working value in the Helm chart: https://github.com/argoproj/argo-helm/blob/168bc63bd6b65586720ebbcea197dbce123c03f5/charts/argo-cd/values.yaml#L2284-L2288

Signed-off-by: Alexandre Gaudreault <[email protected]>
@agaudreault agaudreault requested review from a team as code owners December 3, 2024 02:02
Copy link

bunnyshell bot commented Dec 3, 2024

❌ Preview Environment deleted from Bunnyshell

Available commands (reply to this comment):

  • 🚀 /bns:deploy to deploy the environment

@agaudreault agaudreault changed the title docs(ingress): use GRPC instead of HTTP2 docs(ingress): use GRPC instead of HTTP2 for AWS Dec 3, 2024
@crenshaw-dev crenshaw-dev enabled auto-merge (squash) December 3, 2024 02:04
@crenshaw-dev crenshaw-dev merged commit 205b3d3 into argoproj:master Dec 3, 2024
25 checks passed
adriananeci pushed a commit to adriananeci/argo-cd that referenced this pull request Dec 4, 2024
Signed-off-by: Alexandre Gaudreault <[email protected]>
Signed-off-by: Adrian Aneci <[email protected]>
@agaudreault agaudreault deleted the doc-grpc-svc branch December 6, 2024 04:15
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.

2 participants