Skip to content

Remove session trace restore from bootstrap - #36

Merged
areshand merged 1 commit into
mainfrom
codex/remove-session-s3-read
Aug 25, 2026
Merged

Remove session trace restore from bootstrap#36
areshand merged 1 commit into
mainfrom
codex/remove-session-s3-read

Conversation

@areshand

Copy link
Copy Markdown
Owner

Summary

  • remove the obsolete S3 state download from session container startup
  • keep trace persistence strictly write-only through the deployment sidecar
  • add a runtime contract test preventing S3 restore from returning

Production finding

A fresh GitHub E2E session emitted deterministic s3:GetObject AccessDenied errors because the runtime IAM policy intentionally grants only s3:PutObject. The failure was ignored, but added noise and startup work on every session.

Validation

Not run locally; CI is the validation gate.

@areshand
areshand merged commit e193839 into main Aug 25, 2026
2 of 4 checks passed
@areshand
areshand deleted the codex/remove-session-s3-read branch August 25, 2026 05:47
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