Skip to content

Remove Cam's reingest script#1829

Merged
Oseltamivir merged 1 commit into
mainfrom
cleanup
Jun 18, 2026
Merged

Remove Cam's reingest script#1829
Oseltamivir merged 1 commit into
mainfrom
cleanup

Conversation

@Oseltamivir

@Oseltamivir Oseltamivir commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

Note

Low Risk
Removes a dormant one-off recovery workflow and narrows CI path filters; no runtime product or auth logic changes.

Overview
Removes the manual, workflow_dispatch-only GitHub Action .github/workflows/recover-pr-1798-ingest.yml that re-ingested benchmark results for PR #1798 by validating a failed run-sweep run, reconstructing merge perf-changelog.yaml, reusing artifacts from a prior successful sweep, and dispatching ingest-results to InferenceX-app.

Updates Test Changelog Gate so pushes that only touch perf-changelog.yaml no longer trigger that workflow; changelog-related tests still run when recovery utilities, run-sweep, or other listed paths change.

Reviewed by Cursor Bugbot for commit 10eccfb. Bugbot is set up for automated code reviews on this repo. Configure here.

@Oseltamivir Oseltamivir requested a review from a team June 18, 2026 21:06
@Oseltamivir Oseltamivir merged commit efdff4e into main Jun 18, 2026
4 of 5 checks passed
@Oseltamivir Oseltamivir deleted the cleanup branch June 18, 2026 21:06

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 10eccfb. Configure here.

- main
paths:
- ".claude/commands/recover-failed-ingest.md"
- "perf-changelog.yaml"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changelog gate skips file edits

Medium Severity

Removing perf-changelog.yaml from this workflow’s push path filter means pushes that only change perf-changelog.yaml on non-main branches no longer run Test Changelog Gate, even though the job still validates that file and runs changelog gate pytest suites when it does run.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 10eccfb. Configure here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

1 participant