Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding ability to update a manifest with details from the DLCS #214

Merged
merged 17 commits into from
Jan 15, 2025

Conversation

JackLewis-digirati
Copy link
Collaborator

@JackLewis-digirati JackLewis-digirati commented Dec 19, 2024

Resolves dlcs/protagonist#174

This PR makes it so that manifests are updated from the background handler after they've been completed in the DLCS.

There is 3 new DLCS settings, which are as follows:

"DLCS": {
    "OrchestratorUri": "https://dlcs.digirati.io",
   "ManifestNamedQueryName": "batch-query" // defaults to this
   "OrchestratorDefaultTimeoutMs": 3000 // defautls to this
  }

OrchestratorUri is the base address of the orchestrator and ManifestNamedQueryName is the name of the named query used to retrieve images for a manifest

On top of this, there is an SQS setting for listening to the batch completion queue, and the background handler needs access to the storage bucket:

"AWS": {
    "SQS": {
      "BatchCompletionQueueName": "dlcsspinup-batch-completion-presentation"
    },
    "S3": {
      "StorageBucket": "dlcsspinup-presentation-storage"
    }
  }

Additionally, there is a small change to paintedResource to fill out an asset with an assetId as opposed to canvasPainting. i.e.:

"paintedResources": [
        {
            "type": "PaintedResource",
            "canvasPainting": {
                "canvasId": "https://localhost:7230/26/canvases/w0kyujrecuowii23cztsb3",
                "canvasOrder": 0,
                "thumbnail": "https://dlcs.digirati.io/thumbs/26/18/testAssetByPresentation/full/200,115/0/default.jpg",
                "staticWidth": 6932,
                "staticHeight": 3982
            },
            "asset": {
                "assetId": "https://api.dlcs.digirati.io/customers/26/spaces/18/images/testAssetByPresentation"
            }
        }
    ]

@JackLewis-digirati JackLewis-digirati force-pushed the feature/jobRunnerBatchCompletion branch from ef617e4 to c5ae07b Compare January 9, 2025 16:49
@JackLewis-digirati JackLewis-digirati changed the title initial commit adding ability to update the canvas_paintings table wi… Adding ability to update ta manifest with details from the DLCS Jan 9, 2025
@JackLewis-digirati JackLewis-digirati changed the title Adding ability to update ta manifest with details from the DLCS Adding ability to update a manifest with details from the DLCS Jan 9, 2025
@JackLewis-digirati JackLewis-digirati marked this pull request as ready for review January 13, 2025 16:01
@JackLewis-digirati JackLewis-digirati force-pushed the feature/jobRunnerBatchCompletion branch from d1256c5 to 0d25f9a Compare January 15, 2025 09:52
@JackLewis-digirati JackLewis-digirati merged commit 0b41aa2 into main Jan 15, 2025
4 checks passed
@JackLewis-digirati JackLewis-digirati deleted the feature/jobRunnerBatchCompletion branch January 15, 2025 10:50
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.

Handle HEAD requests
3 participants