Skip to content

codex/fix local driver session token e2e - #3612

Closed
drew wants to merge 1 commit into
codex/fix-podman-credential-host-alias-e2efrom
codex/fix-local-driver-session-token-e2e
Closed

drew wants to merge 1 commit into
codex/fix-podman-credential-host-alias-e2efrom
codex/fix-local-driver-session-token-e2e

Conversation

@drew

@drew drew commented Sep 23, 2026

Copy link
Copy Markdown
Collaborator

Summary

Align the local-driver restart E2E test with split Podman containers and finite launch-scoped session JWTs.

Related Issue

Stability investigation stacked on #3610 and ultimately #3583. The complete local Podman suite found that this test matched both workload and supervisor containers, expected a shell in the capability-free workload, and asserted the legacy non-expiring JWT contract against a finite session token.

Changes

  • Select Podman containers by sandbox name and isolation role; use Podman's archive operation to confirm the workload has no JWT and read the supervisor token.
  • Check that the launch-scoped session JWT expires within one hour, matching the current session-token contract; retain the terminal workload stop assertion.
  • Clarify the legacy and launch-scoped token lifetimes in the gateway architecture documentation.

Testing

  • mise run pre-commit: passed.
  • mise run ci: passed on rerun. The first run hit an unrelated one-off managed_children::tests::child_pid_parser_observes_a_live_child failure; that test passed 100 isolated repetitions and the full rerun.
  • Focused local Podman restart E2E: passed (1 test, 34.13s); CLI conformance passed.
  • Complete local Podman E2E: restart test passed; later corporate proxy tests failed because their gateway-config fixture writes fields into an obsolete table. That separate fixture fix is next in the stack.

Checklist

  • Signed-off conventional commit
  • Documentation updated
  • Full local CI passed
  • Focused Podman E2E passed

Signed-off-by: Drew Newberry <anewberry@nvidia.com>
@drew
drew added this pull request to stack #3585 September 23, 2026 13:59
@copy-pr-bot

copy-pr-bot Bot commented Sep 23, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@drew drew added the automated-flake-find PRs produced by the automated E2E flake-finding campaign label Sep 23, 2026
@drew

drew commented Sep 23, 2026

Copy link
Copy Markdown
Collaborator Author

Closing to narrow the automated flake investigation to the four selected fixes (#3583/#3588, #3598, #3604, #3608). The findings remain available here for later follow-up.

@drew drew closed this Sep 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated-flake-find PRs produced by the automated E2E flake-finding campaign

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant