Skip to content

agentgateway: map frontend TLS annotations to frontend policy#96

Merged
danehans merged 2 commits intomainfrom
issue_59_11
Mar 18, 2026
Merged

agentgateway: map frontend TLS annotations to frontend policy#96
danehans merged 2 commits intomainfrom
issue_59_11

Conversation

@danehans
Copy link
Copy Markdown
Collaborator

@danehans danehans commented Mar 10, 2026

What type of PR is this?
/kind feature
/kind test
/kind documentation

What this PR does / why we need it:

  • adds ingress-nginx provider support for frontend TLS annotations:
    • nginx.ingress.kubernetes.io/ssl-handshake-timeout
    • nginx.ingress.kubernetes.io/ssl-alpn
  • projects parsed values into provider/emitter IR and emits AgentgatewayPolicy.spec.frontend.tls
    for affected HTTPRoutes in the agentgateway emitter
  • adds frontend TLS integration golden input/output fixtures for agentgateway emitter tests
  • adds a frontend TLS e2e scenario and fixtures for the agentgateway e2e suite
  • updates ingress-nginx and agentgateway README docs with mapping details and constraints

Which issue(s) this PR fixes:
Refs #59

Does this PR introduce a user-facing change?:

Added agentgateway frontend TLS projection for ingress-nginx `ssl-handshake-timeout` and `ssl-alpn` annotations.

Parse ingress-nginx ssl-handshake-timeout and ssl-alpn annotations,
carry them through provider and emitter IR, and emit
AgentgatewayPolicy spec.frontend.tls settings for affected routes.

Add integration and e2e test fixtures for frontend TLS policy
projection and update ingress-nginx and agentgateway README
feature documentation.

Refs #59

Signed-off-by: Daneyon Hansen <daneyon.hansen@solo.io>
Emit frontend TLS settings as a dedicated Gateway-targeted
AgentgatewayPolicy so agentgateway validation accepts the resource.
Keep the existing HTTPRoute-scoped policy path for traffic features,
add focused unit coverage for Gateway targeting and conflict detection,
and refresh docs plus golden output to match the new attachment model.

Refs #59

Signed-off-by: Daneyon Hansen <daneyon.hansen@solo.io>
@danehans danehans merged commit a735228 into main Mar 18, 2026
3 checks passed
@danehans danehans deleted the issue_59_11 branch March 18, 2026 03:51
danehans added a commit that referenced this pull request Mar 18, 2026
Ingress NGINX does not document the frontend TLS listener knobs\nthat PR #96 treated as per-Ingress annotations. Related upstream TLS\ncontrols exist, but they do not map cleanly to agentgateway's\nfrontend.tls handshake timeout or ALPN fields.\n\nRemove the unsupported provider and emitter mapping, delete the\nrelated fixtures and tests, and update the ingress-nginx and\nagentgateway docs to reflect that these frontend TLS settings are not\ncurrently projected from ingress-nginx inputs.\n\nRefs #59

Signed-off-by: Daneyon Hansen <daneyon.hansen@solo.io>
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.

1 participant