Skip to content

Fix scheduled transfers not triggering#457

Merged
Dany9966 merged 2 commits into
cloudbase:masterfrom
fabi200123:fix-scheduling-issue
Jun 23, 2026
Merged

Fix scheduled transfers not triggering#457
Dany9966 merged 2 commits into
cloudbase:masterfrom
fabi200123:fix-scheduling-issue

Conversation

@fabi200123

Copy link
Copy Markdown
Contributor

The transfer-cron service started its cron loop in the TransferCronServerEndpoint constructor, which runs in the parent process before oslo_service forks its worker. Since oslo_service forks a worker even with workers=1, the cron loop ran in the parent while the RPC register/unregister handlers ran in the forked child. Each process kept its own in-memory job registry, so schedules created or updated at runtime were added to the child's registry while the parent's loop kept checking an empty one, and scheduled transfers never started.

This was caused by the migration from eventlet to threading, which introduced the parent/child process split for this.

The transfer-cron service started its cron loop in the
TransferCronServerEndpoint constructor, which runs in the parent process
before oslo_service forks its worker. Since oslo_service forks a worker
even with workers=1, the cron loop ran in the parent while the RPC
register/unregister handlers ran in the forked child. Each process kept
its own in-memory job registry, so schedules created or updated at
runtime were added to the child's registry while the parent's loop kept
checking an empty one, and scheduled transfers never started.

This was caused by the migration from eventlet to threading
which introduced the parent/child process split for this.
@fabi200123 fabi200123 force-pushed the fix-scheduling-issue branch 2 times, most recently from 0f79980 to 479041e Compare June 19, 2026 13:31
Comment thread coriolis/tests/integration/transfers/test_schedules.py Outdated
@fabi200123 fabi200123 force-pushed the fix-scheduling-issue branch from 479041e to 5422df0 Compare June 23, 2026 09:04
@Dany9966 Dany9966 merged commit b3b93c8 into cloudbase:master Jun 23, 2026
5 checks passed
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