Skip to content

[bugfix] Retry transient OSError exceptions with the MoviePy backend#542

Merged
Breakthrough merged 3 commits intomainfrom
issue-496-deflake-moviepy-tests
Apr 20, 2026
Merged

[bugfix] Retry transient OSError exceptions with the MoviePy backend#542
Breakthrough merged 3 commits intomainfrom
issue-496-deflake-moviepy-tests

Conversation

@Breakthrough
Copy link
Copy Markdown
Owner

@Breakthrough Breakthrough commented Apr 19, 2026

This isn't really a fix for #496 per-se, but it should greatly reduce how many builds fail because of this issue. It works around an issue where sometimes on CI builders we see failures when opening/seeking/resetting videos due to pipe races on slow or heavily loaded systems.

Two fixes:

  1. Add retry with backoff for the MoviePy backend to mitigate this in production
  2. Deflake unit tests themselves

@Breakthrough Breakthrough force-pushed the issue-496-deflake-moviepy-tests branch from 9875b01 to 71ce5a8 Compare April 19, 2026 22:35
@Breakthrough Breakthrough self-assigned this Apr 19, 2026
@Breakthrough Breakthrough changed the title [tests] Add deflake for #496 [bugfix] Retry transient OSError exceptions with the MoviePy backend Apr 19, 2026
@Breakthrough Breakthrough linked an issue Apr 19, 2026 that may be closed by this pull request
@Breakthrough Breakthrough force-pushed the issue-496-deflake-moviepy-tests branch from a9d74b8 to b28460a Compare April 19, 2026 23:05
@Breakthrough Breakthrough merged commit e67b0a3 into main Apr 20, 2026
28 checks passed
@Breakthrough Breakthrough deleted the issue-496-deflake-moviepy-tests branch April 20, 2026 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MoviePy Backend Can Result In Transient OSError

1 participant