Skip to content

Include token broker lifecycle in fleet simulation stress tests - #20043

Merged
pelikhan merged 3 commits into
mainfrom
copilot/ensure-token-broker-in-stress-tests
Oct 31, 2026
Merged

pelikhan merged 3 commits into
mainfrom
copilot/ensure-token-broker-in-stress-tests

Conversation

Copilot AI commented Oct 31, 2026 •

Copy link
Copy Markdown
Contributor

The fleet simulation omitted token brokering from its review lifecycle. This change gates each dispatched batch on a production broker exchange and invalidates its credential after all reviews complete.

  • Lifecycle coverage

    • Require one mint and revocation per batch.
    • Check repository-scoped permissions, throttling, idempotent invalidation, cleanup isolation, and remint denial.
    • Include broker delays in the existing one-hour review SLA and expose lifecycle counters in fleet-stress.json.
  • Isolated boundaries

    • Use synthetic job identities, a private fixture policy, ephemeral App keys, and an in-process issuer transport.
    • Leave production authority, rate limits, and context ceilings unchanged; document that OIDC, hosted transport, and Key Vault remain outside coverage.
  • Execution budget

    • Increase the stress timeout to 60 minutes and workflow timeout to 65 minutes for the added broker workload.

Copilot AI and others added 2 commits October 31, 2026 20:12
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…limit

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan

Copy link
Copy Markdown
Collaborator

@copilot Fix ci failures

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot AI commented Oct 31, 2026

Copy link
Copy Markdown
Contributor Author

@copilot Fix ci failures

Fixed in fb23950: relay tests now use test-scoped subprocess contexts, and both agent catalog artifacts are regenerated from the dashboard source. The failed tooling area passes (596 passed, 1 skipped); full Go formatting/lint, relay tests, Go vet and root lint pass. New CI results are pending after the push.

@github-actions

Copy link
Copy Markdown
Contributor
Dashboard dead views

Dead element views (0):
Dead dashboard pages (0):
Analyzed 8 element renderers and 70 dashboard pages across 1 dashboard.json file(s) from dashboard/site/src/components/ui-elements.js.

⚠️ This reachability report is informational and does not block the pull request.
🔎 View the workflow run and report artifact.

@pelikhan
pelikhan merged commit 3570599 into main Oct 31, 2026
50 checks passed
@pelikhan
pelikhan deleted the copilot/ensure-token-broker-in-stress-tests branch October 31, 2026 22:20
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