Skip to content

fix(ci): fix CI PR cleanup job to delete in batches - #587

Merged
ndizazzo merged 2 commits into
mainfrom
fix/ci-pr-cleanup
May 19, 2026
Merged

fix(ci): fix CI PR cleanup job to delete in batches#587
ndizazzo merged 2 commits into
mainfrom
fix/ci-pr-cleanup

Conversation

@ndizazzo

Copy link
Copy Markdown
Collaborator

Job from @IvGolovach failed on cleanup due to rate limits, this should address it

@ndizazzo ndizazzo self-assigned this May 19, 2026
Copilot AI review requested due to automatic review settings May 19, 2026 05:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds batching with pauses between deletions in the PR cleanup workflow to avoid hitting GitHub's secondary rate limits during cache/artifact cleanup.

Changes:

  • Introduces a deleteBatchSize (20) and deleteBatchPauseMs (15s) pause between batches for cache deletions.
  • Adds a deleteInBatches helper for artifact deletions and collects artifacts before deleting in batches.
  • Emits a core.notice between batches indicating progress and pause duration.

@michaelneale

Copy link
Copy Markdown
Collaborator

Is the swift failure related to this?

@ndizazzo

Copy link
Copy Markdown
Collaborator Author

Is the swift failure related to this?

Nah - it's because an earlier step didn't produce an artifact to do smoke tests with. I'll throw a fix for that in with this

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@ndizazzo
ndizazzo merged commit 3487b98 into main May 19, 2026
20 checks passed
@ndizazzo
ndizazzo deleted the fix/ci-pr-cleanup branch May 19, 2026 06:53
michaelneale added a commit that referenced this pull request May 20, 2026
* origin/main:
  fix(mesh): skip filtered peers in gossip dial loop to unwedge `--auto` (#602)
  docs(agents): clarify just build vs release-build for serious testing (#599)
  build: ozempic — slim binary -42 MB / -47 MB (#592)
  fix(runtime): proxy through mesh during serve --auto startup (#591)
  chore(code-quality): Set max 200 line limit for long methods (#595)
  fly size bump (#590)
  agents is now up to date (#588)
  fix(ci): fix CI PR cleanup job to delete in batches (#587)
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.

3 participants