Skip to content

test(server): stabilize JWKS validation tests - #3225

Merged
pimlock merged 1 commit into
NVIDIA:mainfrom
grs:jwks_validation_test_fixes
Sep 8, 2026
Merged

test(server): stabilize JWKS validation tests#3225
pimlock merged 1 commit into
NVIDIA:mainfrom
grs:jwks_validation_test_fixes

Conversation

@grs

@grs grs commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Prevents intermittent failures from the jwks_validation tests.

Related Issue

No issue required: just a test fix

Changes

  • Keep Wiremock servers alive for the full lifetime of JWKS cache tests, preventing pooled servers from being recycled prematurely.
  • Replace real-time sleeps in the stale-key eviction test with explicit cache timestamp manipulation.
  • Increase the test TTL window to prevent scheduler delays from crossing grace-period boundaries.
  • Verify that the first unknown-kid lookup performs a JWKS refresh.
  • Place subsequent unknown-kid lookups deterministically within the cooldown window.
  • Preserve the existing assertions for JWKS refresh throttling, stale-key retention, and eventual eviction.

Testing

  • mise run pre-commit passes
  • Unit tests added/updated
  • E2E tests added/updated (if applicable)

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable)

Signed-off-by: Gordon Sim <gsim@redhat.com>
@grs
grs requested review from a team, derekwaynecarr, mrunalp and sjenning as code owners September 8, 2026 19:33
@copy-pr-bot

copy-pr-bot Bot commented Sep 8, 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.

@pimlock

pimlock commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

/ok to test 31f5fcc

@pimlock
pimlock enabled auto-merge September 8, 2026 20:52
@pimlock
pimlock added this pull request to the merge queue Sep 8, 2026
Merged via the queue into NVIDIA:main with commit d4f4b70 Sep 8, 2026
57 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