advances pulibrary/figgy#6329 (close once reindex is done)
supercedes #1529 -- I thought I could push the valid? check down the stack, but the job won't run if the resource hasn't persisted, and failed validation can keep a resource from persisting. I didn't want to rewrite the reindex jobs to take a url instead of a resource, so this PR still proposes recording the job as a failure, after some retries. It also sends more details to honeybadger.
advances pulibrary/figgy#6329 (close once reindex is done)
supercedes #1529 -- I thought I could push the
valid?
check down the stack, but the job won't run if the resource hasn't persisted, and failed validation can keep a resource from persisting. I didn't want to rewrite the reindex jobs to take a url instead of a resource, so this PR still proposes recording the job as a failure, after some retries. It also sends more details to honeybadger.