Skip to content

Bundle rollout attachments into a gzip tar archive - #50446

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/973da6eec2249b04ef7992b5e1e4fd27f044b80f
Oct 2, 2026
Merged

copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/973da6eec2249b04ef7992b5e1e4fd27f044b80f

Conversation

@copyberry

@copyberry copyberry Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

Bundle rollout attachments into a gzip tar archive

What changed

  • Combine file-backed rollouts and buffered rollout prefixes into rollouts.tar.gz, keeping diagnostics as separate attachments.
  • Bound the archive and its upload envelope by the existing size limit, skip invalid or duplicate archive filenames, and fall back to individual rollout attachments if archiving fails.
  • Add rollout_archive_filename to the report thread index so rollout filenames identify archive entries or fallback attachments.

Testing

Add an upload test that extracts the archive and verifies the filenames and contents of both a compressed file-backed rollout and a buffered rollout. Update the thread index test to check the archive filename.

## What changed

- Combine file-backed rollouts and buffered rollout prefixes into `rollouts.tar.gz`, keeping diagnostics as separate attachments.
- Bound the archive and its upload envelope by the existing size limit, skip invalid or duplicate archive filenames, and fall back to individual rollout attachments if archiving fails.
- Add `rollout_archive_filename` to the report thread index so rollout filenames identify archive entries or fallback attachments.

## Testing

Add an upload test that extracts the archive and verifies the filenames and contents of both a compressed file-backed rollout and a buffered rollout. Update the thread index test to check the archive filename.

GitOrigin-RevId: 973da6eec2249b04ef7992b5e1e4fd27f044b80f
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/973da6eec2249b04ef7992b5e1e4fd27f044b80f branch from f4b5f27 to 9b0a676 Compare October 2, 2026 22:48
@copyberry
copyberry Bot merged commit 9b0a676 into main Oct 2, 2026
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/973da6eec2249b04ef7992b5e1e4fd27f044b80f branch October 2, 2026 22:48
@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@github-actions github-actions Bot locked and limited conversation to collaborators Oct 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant