yalelibrary / YUL-DC

Preliminary issue tracking for Yale University Libraries Digital Collections project
3 stars 0 forks source link

Fix ParentObject.replace_preservica_tif(co) #2864

Open martinlovell opened 1 week ago

martinlovell commented 1 week ago

ParentObject.replace_preservica_tif(co) copies all images to the same child OID. The last image is the one that ends up in child OID in the /data directory for all children.

See: https://github.com/yalelibrary/yul-dc-management/pull/1403/files#diff-acc421bf5477cf1667d0b61e9ded247f3b42335a2425020920d8c20919ff0384L293

martinlovell commented 1 week ago

https://github.com/yalelibrary/yul-dc-management/pull/1403

K8Sewell commented 1 week ago

Deployed to Test, Demo, and UAT with release v2.70.2 - however, UAT is connected to the Test Preservica instance rather than prod and the info object ending in fff5f5 is only available on prod.

Do we have an object in test preservica with some duplicates we can try this out on?

sshetenhelm commented 1 week ago

To clarify, do you mean an object ingested into TEST Preservica that has two copies of the exact same image, with the exact same filename?

sshetenhelm commented 1 week ago

Just asked our Digital Studio to ingest this object in TEST Preservica. I'll update when they do.

sshetenhelm commented 1 week ago

'Create parent' batch process in progress -- https://collections-uat.library.yale.edu/management/batch_processes/2052

sshetenhelm commented 6 days ago

Object uploaded successfully. Please note the repository added more files to this object, so it will now display as having 64 children instead of the original 38. Waiting on results of resync process now.

sshetenhelm commented 6 days ago

Successful? No longer running, unique children still displaying -- https://collections-uat.library.yale.edu/management/batch_processes/2055/parent_objects/901624363

Do we need to run any other tests for this?

K8Sewell commented 6 days ago

Testing with this object was the main thing we needed to check. Unless Martin has other test scenarios in mind - we are set.

martinlovell commented 6 days ago

We are all set as long as re-sync works with the right images in the right place.

sshetenhelm commented 4 days ago

Stakeholder stated they expected hundreds more images in this object. However, after investigating, we think this may be a stakeholder-to-Preservica workflow issue rather than a Preservica-to-DCS syncing issue. Let's keep this ticket and the next open for a little longer until we hear back from the stakeholder again.

sshetenhelm commented 3 days ago

Reconstructed Preservica object ingesting now -- https://collections-uat.library.yale.edu/management/batch_processes/2067

sshetenhelm commented 2 days ago

"New" object with 942 unique images successfully created. Pressed the "resync" button; we'll see what happens.