Skip to content
Snippets Groups Projects

Use a worker for S3 imports

Merged Erwan Rouchet requested to merge s3-import-worker into master
All threads resolved!

Closes #1795 (closed)

  • I removed ProcessBuilder.build_iiif, a leftover of #1302 (closed).
  • I made ProcessBuilder._build_task assign the ARKINDEX_WORKER_RUN_ID based on Task.worker_run_id, to avoid duplicating code or forgetting to set the environment correctly.
  • The system checks wiki page has been updated.

This needs to be tested with arkindex/workers/import!4

Merge request reports

Checking pipeline status.

Approval is optional

Merged by Bastien AbadieBastien Abadie 9 months ago (Jul 4, 2024 7:37am UTC)

Merge details

  • Changes merged into master with 58b0cdcc (commits were squashed).
  • Deleted the source branch.
  • Auto-merge enabled

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Bastien Abadie
  • I got a successful run with updated import worker arkindex/workers/import!4

  • Bastien Abadie added 16 commits

    added 16 commits

    • bdf7e486...a0f6ced5 - 7 commits from branch master
    • 4ba638a7 - Add docker.ingest_image setting
    • e488d1b2 - Add WorkerVersion.objects.ingest_version
    • 1b521acb - Add system check
    • da265a6a - Remove unused code from IIIF imports
    • 9baa524b - Automatically set worker run ID env from Task.worker_run
    • 1a21f99f - Use S3 import worker when building S3 process
    • 1174a0d9 - Remove unnecessary configuration hashing
    • ddd53b78 - Update retry test
    • fdbf0038 - Update S3 tests

    Compare with previous version

  • Bastien Abadie resolved all threads

    resolved all threads

  • Bastien Abadie enabled an automatic merge when the pipeline for fdbf0038 succeeds

    enabled an automatic merge when the pipeline for fdbf0038 succeeds

  • Please register or sign in to reply
    Loading