Closed mikeapp closed 5 months ago
Check in with Amanda about parallel QS work.
We're on 2.7.7 it seems.
I drafted a PR to upgrade our phusion passenger image to 3.1. Will await information from Amanda to finalize the Ruby version we can jump to. https://github.com/yalelibrary/yul-dc-camerata/pull/365
Amanda confirms that there are no conflicts between Ruby 3.2.0 and Blacklight 7.33.1, we can target 3.2.0.
PRs ready for review: Camerata - https://github.com/yalelibrary/yul-dc-camerata/pull/365 Management - In progress (working through a build failure) Blacklight - In progress (same)
Working through these errors to fix the failing tests on management
Set aside till the multiple jobs issues is resolved
Failing tests after upgrade started at 457 but we're now (as of 12/14/23) at 10.
Time spent so far - 8 weeks Remaining work:
With most recent commit we sit at about 150 failing tests. They fail for 3 main reasons: objects are not persisting, test not recognizing log in of user, and an unexpected alert. The alert has an easy fix and the other two issues are still in progress to determine the cause of the failures.
Tickets created to fix remaining broken specs:
PRs ready for review:
Camerata - https://github.com/yalelibrary/yul-dc-camerata/pull/365
Management - https://github.com/yalelibrary/yul-dc-management/pull/1299
Blacklight - in another ticket
Waiting on test plan
Test environment is live with Management v2.67.2 so this is ready to be tested.
There may be some issues between Preservica and DCS that need to be further investigated. Will do more testing and report back.
Other aspects of Management look good.
Preservica upload is not working in UAT. Unsure if it is due to this bug fix or the Ruby upgrade.
When attempting to upload, I received the following error:
Child Object 901622792 failed to convert PTIFF due to Expected file originals/92/90/16/22/79/901622792.tif not found.
So I guess the child image did not successfully copy over from Preservica?
Video
Will also comment in Preservica bug ticket.
Great news! Got an object to go through UAT -- not the checksum test one, but a different one, so likely the checksum one is still causing errors but maybe the non-checksum errors are fixed? Batch process -- https://collections-uat.library.yale.edu/management/batch_processes/1881/parent_objects/901623825
I will test again before we officially celebrate.
...but these two failed: https://collections-uat.library.yale.edu/management/batch_processes/1882
I don't know why I received the image error for these two but not for the above.
Story
Move to Ruby 3, perhaps https://hub.docker.com/r/phusion/passenger-ruby30 or later image?
I'm confused about this earlier PR: https://github.com/yalelibrary/yul-dc-camerata/pull/356 , it seems to leave us on 2.7?
Acceptance