[RTADPA BW tests] Small cleanups after !359
FYI @lugrazet
Goes with LHCbPR2HD!271
This MR just cleans up the logging in the bandwidth tests a little bit after !359 (merged), and grabs the latest HLT2 output from the current_hlt2_output
directory on /eos/
. I think it's neat to have all the transient files we need to live in one directory.
I also checked while here that the download of files for the BW test does not happen if the file is already present, as we thought. The log should now give us more info on whether the download happens or not, so we can spot this more easily in the future.
Merge request reports
Activity
assigned to @rjhunter
mentioned in merge request lhcb-core/LHCbPR2HD!271 (merged)
requested review from @lugrazet
- Resolved by Ross John Hunter
Hi @rjhunter thanks for this it looks good (apologies I accidentally closed the MR
).I have one curiosity. When you say "The log should now give us more info on whether the download happens or not, so we can spot this more easily in the future." I assume you're referring to just inferring that we didn't download when
Finished file downloads. This took: {datetime.now() - before_copy}"
prints out as close to 0s. Is this correct?
mentioned in merge request lhcb-core/LHCbPR2HD!270 (merged)
mentioned in merge request !361 (merged)
mentioned in commit f4c873bf
mentioned in commit 03521d8f
changed title from [RTA BW tests] Small cleanups after !359 (merged) to [RTADPA BW tests] Small cleanups after !359 (merged)