Skip to content
Snippets Groups Projects

[RTADPA BW Tests] Allow uploading of .root files

Merged Luke Grazette requested to merge lugrazet-bwtests-uploadhistos into master
All threads resolved!

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

Pipeline #10453307 passed

Pipeline passed for 219cdd85 on lugrazet-bwtests-uploadhistos

Merged by Luke GrazetteLuke Grazette 2 months ago (Feb 18, 2025 1:22pm UTC)

Loading

Pipeline #10458395 passed

Pipeline passed for 16025569 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Luke Grazette added 1 commit

    added 1 commit

    Compare with previous version

  • Luke Grazette reset approvals from @rjhunter by pushing to the branch

    reset approvals from @rjhunter by pushing to the branch

  • Ross John Hunter approved this merge request

    approved this merge request

  • Luke Grazette resolved all threads

    resolved all threads

  • Luke Grazette mentioned in commit 16025569

    mentioned in commit 16025569

  • mentioned in merge request lhcb/Moore!4345 (closed)

  • Author Developer

    Confirmed (with a ci-test on lhcb/Moore!4345 (closed)) this indeed published the .root file as expected :slight_smile:

  • Please register or sign in to reply
    Loading