[RTADPA BW Tests] Allow uploading of .root files
We've been requested to allow the functionality to upload the monitoring histograms from a BW Test on command.
-> Soln: lhcb/Moore!4345 (closed) can be included in ci-test
on command.
Locally this works as expected and makes the webpage link to what the correct location on eos would be... however... the Handler only allows certain file extensions to be upload.
This MR updates that list to include .root
files.
- I checked that locally the only files under
BASEDIR/Output/*.root
are these new histos so shouldn't accidentally be uploaded something we previously ignored. - so with this merged the histos would be uploaded upon request i.e. when !4345 is included. Thus RTA/DPA are happy and can include Moore!4345 whenever they wish.
Merge request reports
Activity
added DPA-WP1 label
requested review from @rjhunter
assigned to @lugrazet
- Resolved by Luke Grazette
Perhaps we open an issue to remind us to take this out of the allowed list of file extensions once we are more centrally handling the histograms (i.e. James' work)? That way we don't leave open what is really just a temporary hack
- Resolved by Luke Grazette
reset approvals from @rjhunter by pushing to the branch
mentioned in commit 16025569
mentioned in merge request lhcb/Moore!4345 (closed)
Confirmed (with a ci-test on lhcb/Moore!4345 (closed)) this indeed published the
.root
file as expected