Skip to content

feat(helm): add agentgateway ingress support - #3714

Open
danehans wants to merge 11 commits into
NVIDIA:mainfrom
danehans:2469-agentgateway-ingress/danehans
Open

danehans wants to merge 11 commits into
NVIDIA:mainfrom
danehans:2469-agentgateway-ingress/danehans

Conversation

@danehans

@danehans danehans commented Sep 25, 2026 •

Copy link
Copy Markdown

Summary

Add agentgateway as a supported Kubernetes Gateway API ingress controller using a dedicated, chart-owned Gateway. Preserve HTTPS when the CLI establishes SSH forwarding through an externally terminated TLS endpoint.

Related Issue

Fixes: #2469
Fixes: #3674

Follow-up: #3715 tracks shared, platform-managed agentgateway Gateway support.

Changes

  • Create a dedicated agentgateway Gateway and GRPCRoute through the existing chart model.
  • Reject agentgateway Gateway names that conflict with the OpenShell Service.
  • Add dedicated frontend TLS, backend TLS re-encryption, and HA coverage.
  • Add Helm tests, Kubernetes E2E tasks, and CI jobs for the dedicated topology.
  • Document dedicated deployment and troubleshooting workflows.
  • Preserve the external HTTPS scheme for SSH forwarding.

Testing

  • mise run pre-commit
  • mise run helm:test
  • mise run helm:lint
  • mise run e2e:kubernetes:agentgateway-tls
  • mise run e2e:kubernetes:agentgateway-backend-tls
  • mise run e2e:kubernetes:agentgateway

The local-image registry regression required OPENSHELL_REGISTRY=docker.io/openshell for local E2E and is tracked separately in #3696 and PR #3701.

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (not applicable; user and contributor documentation updated)

Fixes: NVIDIA#2469
Signed-off-by: Daneyon Hansen <daneyon.hansen@solo.io>
Add shared and dedicated TLS, backend re-encryption, HA validation, and the
related chart documentation and E2E workflows.

Fixes: NVIDIA#2469
Signed-off-by: Daneyon Hansen <daneyon.hansen@solo.io>
Resolve the external scheme together with its authority and cover the
frontend TLS relay path in shared and dedicated agentgateway E2E tasks.

Fixes: NVIDIA#3674
Signed-off-by: Daneyon Hansen <daneyon.hansen@solo.io>
Run the base agentgateway Kubernetes job with the non-HA feature set so the reusable workflow does not suppress the Nextest suite.

Fixes: NVIDIA#2469
Signed-off-by: Daneyon Hansen <daneyon.hansen@solo.io>
@copy-pr-bot

copy-pr-bot Bot commented Sep 25, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Signed-off-by: Daneyon Hansen <daneyon.hansen@solo.io>
Signed-off-by: Daneyon Hansen <daneyon.hansen@solo.io>
@danehans
danehans marked this pull request as draft September 25, 2026 16:16
Signed-off-by: Daneyon Hansen <daneyon.hansen@solo.io>
Signed-off-by: Daneyon Hansen <daneyon.hansen@solo.io>
Signed-off-by: Daneyon Hansen <daneyon.hansen@solo.io>
@danehans

Copy link
Copy Markdown
Author

Updated the implementation to use a dedicated, chart-owned agentgateway Gateway. Shared Gateway and ListenerSet support moved to #3715, with the prototype preserved at https://github.com/danehans/OpenShell/tree/3715-shared-agentgateway-gateway/danehans.

Signed-off-by: Daneyon Hansen <daneyon.hansen@solo.io>
Signed-off-by: Daneyon Hansen <daneyon.hansen@solo.io>
@danehans
danehans marked this pull request as ready for review September 25, 2026 18:46

This branch has not been deployed

No deployments
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.

bug: SSH forwarding drops HTTPS behind TLS termination feat(helm): support agentgateway ingress

1 participant