Skip to content

ci(review): keep the review debug bundle for 30 days (#4742) - #3540

Merged
inkeep-oss-sync[bot] merged 1 commit into
mainfrom
copybara/sync
Sep 19, 2026
Merged

inkeep-oss-sync[bot] merged 1 commit into
mainfrom
copybara/sync

Conversation

@inkeep-oss-sync

Copy link
Copy Markdown
Contributor

Copybara-translated 1 Inkeep OSS change. Rebase-merge this PR so the prepared commit lands directly on public main.

The per-run debug upload is the only place a finished review's own
evidence survives: its execution output, the CLI debug log, the PR
context it was given, and the review manifests. Nothing else retains
them, so once the artifact expires a round can no longer be audited at
all. At 7 days it was expiring before anyone went looking, which is the
window this widens.

30 sits under the 90-day maximum every repository these workflows run in
allows, so the value is stored as written. Above that maximum the upload
action does not fail; it quietly substitutes the maximum and logs a
warning on every run, which would leave the workflow claiming a window it
does not have.

All three review workflows move together, and a test now pins that they
agree on one value, since the root and its two mirrors are edited
separately and drift between them is silent.

GitOrigin-RevId: 801018febc2e83d139e49050a65116e56734b457
@changeset-bot

changeset-bot Bot commented Sep 19, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 7f88820

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@inkeep-oss-sync
inkeep-oss-sync Bot merged commit 3b74762 into main Sep 19, 2026
@inkeep-oss-sync
inkeep-oss-sync Bot deleted the copybara/sync branch September 19, 2026 01:29
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