search

bihealth / sodar-server

SODAR: System for Omics Data Access and Retrieval
https://github.com/bihealth/sodar-server
MIT License
14 stars 3 forks source link

Add logging for BatchCalculateChecksumTask exceptions #1940

Closed mikkonie closed 2 months ago

mikkonie commented 2 months ago

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.

mikkonie commented 2 months ago

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.