Skip to content

fix: ignore_cleanup_errors to mitigate strange flakes in sentry CI#312

Merged
joshuarli merged 1 commit intomainfrom
fix/mitigate-spurious-cleanup
Apr 27, 2026
Merged

fix: ignore_cleanup_errors to mitigate strange flakes in sentry CI#312
joshuarli merged 1 commit intomainfrom
fix/mitigate-spurious-cleanup

Conversation

@joshuarli
Copy link
Copy Markdown
Member

@joshuarli joshuarli commented Apr 27, 2026

edit: here's a failing job https://github.com/getsentry/sentry/actions/runs/25004348752/job/73222906040 which is using runner version 20260426.100

https://github.com/actions/runner-images/releases/tag/ubuntu24/20260426.100

git got upgraded to 2.54.0

claude says: Git 2.54.0's geometric repacking default means
gc.auto (which fires in the background after fetch/clone) now creates more
files in .git/objects/pack/ — MIDX layers, bitmap files, incremental packs
— compared to 2.53.0. If this background gc process creates any of those
files after Python's shutil.rmtree has already scanned the directory but
before the final os.rmdir('.git'), you get ENOTEMPTY. The race window is
small, hence the flakiness rather than consistent failure.


not really sure what's going on here but my best guess is github upgraded/reconfigured git on their runner images to do more background repo bookkeeping stuff and that's racing with python's tempdir cleanup

mitigate this; it's not a symptom of anything bad happening, and it's fine if the dirs aren't completely cleaned up since it's part of an ephemeral job anyway

@joshuarli joshuarli merged commit 92999b5 into main Apr 27, 2026
14 checks passed
@joshuarli joshuarli deleted the fix/mitigate-spurious-cleanup branch April 27, 2026 22:15
joshuarli added a commit to getsentry/sentry that referenced this pull request Apr 28, 2026
still messed up here:
https://github.com/getsentry/sentry/actions/runs/25064614491/job/73428196070

was hoping getsentry/devservices#312 would have
fixed things but nope

this is the more nuclear option
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.

2 participants