Skip to content
This repository was archived by the owner on Apr 4, 2026. It is now read-only.

fix(backfill): fix docker compose timing and add worker improvements#71

Merged
christopherwxyz merged 5 commits into
mainfrom
fix/backfill-docker-compose
Jan 1, 2026
Merged

fix(backfill): fix docker compose timing and add worker improvements#71
christopherwxyz merged 5 commits into
mainfrom
fix/backfill-docker-compose

Conversation

@christopherwxyz

Copy link
Copy Markdown
Contributor

Summary

  • Fix docker-compose.backfill.yml missing depends_on for worker (caused instant completion)
  • Change condition from service_started to service_completed_successfully (fixes race condition)
  • Add --exit-on-complete and --idle-timeout flags to worker
  • Add BACKFILL_WORKERS env var for scaling via deploy.replicas
  • Slim down README

Changes

Docker Compose:

  • Worker now waits for queue to finish populating before starting
  • Workers exit cleanly when backfill completes (queue empty for 60s)
  • Scale workers with --scale backfill-worker=4 or BACKFILL_WORKERS=4

Worker CLI:

--exit-on-complete    Exit when queue is empty (default: false)
--idle-timeout <secs> Seconds to wait before exiting (default: 30)

- Fix docker-compose.backfill.yml missing depends_on for backfill-worker
- Change depends_on condition from service_started to service_completed_successfully
  to prevent race condition where worker starts before queue finishes populating
- Add --exit-on-complete flag to worker to exit cleanly when backfill completes
- Add --idle-timeout flag to configure how long to wait before considering complete
- Add deploy.replicas support for scaling workers via BACKFILL_WORKERS env var
- Update README, development.md, and docker-hub.md with new backfill options
@christopherwxyz
christopherwxyz merged commit 7af3b46 into main Jan 1, 2026
2 checks passed
@christopherwxyz
christopherwxyz deleted the fix/backfill-docker-compose branch January 1, 2026 18:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant