search

sul-dlss / common-accessioning

Suite of robots that handle the tasks of accessioning digital objects
Other
2 stars 1 forks source link

update Nokogiri on results to Nokogiri Reader #1318

Closed dnoneill closed 3 months ago

dnoneill commented 3 months ago

Why was this change made? 🤔

Fixes #1308

How was this change tested? 🤨

âš¡ âš  If this change has cross service impact, including writes to shared file systems, run integration tests and/or test in [stage|qa] environment, in addition to specs. âš¡

peetucket commented 3 months ago

Has this been tested on qa? i.e.

  1. deploy the branch to qa
  2. run the integration tests, verify it works

We can then also try to reset the step on the really big object that previously failed while deployed

dnoneill commented 3 months ago

It currently is on QA and I am running it on https://argo-qa.stanford.edu/view/druid:gq110sz4835. It seems to be taking a while but I am pretty sure that is on the abbyy side because the files don't seem to exist yet, there is nothing in the output folder. Figured I would check at the end of the day to see if things have cleared up.

peetucket commented 3 months ago

Ok, thanks. Let's HOLD this PR until we see if this is anything else to be done.