Skip to content

charts: set the egress credential cache TTL per extra provider - #170

Merged
codyhartsook merged 1 commit into
mainfrom
codyhartsook/cacheTtl
Nov 13, 2026
Merged

codyhartsook merged 1 commit into
mainfrom
codyhartsook/cacheTtl

Conversation

@codyhartsook

@codyhartsook codyhartsook commented Nov 11, 2026 •

Copy link
Copy Markdown

Some credential providers hand out a user's token only while the actor is handling that user's request. The egress gateway caches each fetched credential for 5 minutes, and a cache hit never reaches the provider. So after the request ends, the actor can keep calling the upstream as that user for up to 5 minutes. A grant the provider revokes lags the same way.

agentgateway/agentgateway#4170 added a per-provider cacheTtl, and the pinned v1.7.0 image has it. This PR exposes it on atenetEgress.extraCredentialProviders:

atenetEgress:
  extraCredentialProviders:
  - name: vault.example.com
    address: vault-provider.vault.svc:50051
    cacheTtl: 0s
  • Unset keeps the 5m default and renders the same config as before.
  • 0s sends every request to the provider.
  • A bad duration or an unknown key like cacheTTL fails at render time. extraCredentialProviders hasn't shipped in a release, so no existing values break.

The bundled k8s.io provider and ate-setup keep the default. ate-setup also drives the Envoy dataplane, which doesn't cache.

AI use: AI-assisted. I reviewed the change, ran helm unittest charts/substrate, and loaded the rendered config with agentgateway --validate-only on the pinned image.

  • Tests pass
  • Docs updated

@codyhartsook
codyhartsook marked this pull request as ready for review November 11, 2026 20:15
The egress gateway reuses a fetched credential for 5 minutes, and a cache
hit never reaches the provider. A grant the provider revokes keeps working
until the entry expires. agentgateway/agentgateway#4170 added a per-provider
cacheTtl, so pass it through atenetEgress.extraCredentialProviders.

Entries with unknown keys now fail the render. Before, the chart dropped a
misspelled cacheTtl and the 5m cache stayed on.

Signed-off-by: Cody Hartsook <cody.hartsook@solo.io>
@codyhartsook
codyhartsook merged commit f0c5bcf into main Nov 13, 2026
8 checks passed
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