I'm not convinced this is the right fix.

At the point where this change is made, we are iterating over the local var logs. This dictionary of log info is only populated by:

• log files found within resultdir (which can only be present if it is already created)
• the mock log and its corresponding ts log

The latter can be present before resultdir is created by mock, however neither the mock log nor its ts file should be in resultdir. The code here is not handling the case correctly.

Shortest fix is probably to use workdir for all the ts.log files. This might be more correct too since they are generated by koji, not mock.

Alternately, we could fix the code in the logs.items loop to choose the correct ts file path in the mocklog case.

updated to workdir

lgtm

Commit 81767d09 fixes this pull-request