dgrnbrg/autotest — comment on commit cd4da65bb0f7cdad505bd1f97ca18ba6e1f5e6ee
File: server/hosts/logfile_monitor.py line 90
By dgrnbrg:

I don't understand where you're suggesting to put the lock.

This code is thread-unsafe due to the implementation of host.job.record, and there's no way to make it threadsafe without other serious ramifications. job.record is a convoluted code path that is too difficult to verify.

I think that the cost of potential races should be weighed against the benefit of massively increased test startup performance.