bihealth / sodar-server

SODAR: System for Omics Data Access and Retrieval

Home Page:https://github.com/bihealth/sodar-server

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Add logging for BatchCalculateChecksumTask exceptions

mikkonie opened this issue · comments

Currently, we recover gracefully for checksum calculation errors in BatchCalculateChecksumTask. These errors can be caused by problems in the storage system used for iRODS resource servers (and we have plenty of problems with the one we currently use in production).

This is nice, but we don't currently log these errors in any way, simply pass and move on to the next file. The entire validation/moving operation will of course eventually fail as the checksum was not correctly computed.

As the first step, we should at least add error logging for this step so these issues can be monitored better.

Never mind, the logging was already recently added, I was just looking at the wrong output. Mental note: don't open issues before the 1st morning coffee.