Skip to content
Snippets Groups Projects

ATR-28571: Extend Trig_reco_tf.py functionality to allow athenaHLT running with legacy release inside a container

Merged ATR-28571: Extend Trig_reco_tf.py functionality to allow athenaHLT running with legacy release inside a container
All threads resolved!
Merged Oleg Kuprash requested to merge okuprash/athena:trig_reco_cont into 24.0
All threads resolved!

For debug stream recovery, current ART test test_trigP1_v1PhysP1_trfDbgStream_build.py is processing in Rel. 24 a data21 file taken with Rel. 22.0.35. To allow rerunning Trigger step with 22.0.35 in a transform substep, a centos7 container has to be invoked. This MR extends container functionality of Trig_reco_tf.py in a similar way to Reco_tf.py.
Tagging @mark, @jcurran, @wiedenma

Merge request reports

Pipeline #6966881 passed

Pipeline passed for 1eccc042 on okuprash:trig_reco_cont

Merged by Duc TaDuc Ta 1 year ago (Feb 27, 2024 5:13pm UTC)

Merge details

  • Changes merged into 24.0 with f88d9b30.
  • Did not delete the source branch.

Activity

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

    added 1 commit

    • 1eccc042 - Use legacy config for debug recovery of data23 and older

    Compare with previous version

  • Mark Stockton resolved all threads

    resolved all threads

  • This merge request affects 2 packages:

    • HLT/Trigger/TrigTransforms/TrigTransform
    • Trigger/TrigValidation/TrigP1Test

    Affected files list will not be printed in this case

    Adding @vmartin ,@fpastore ,@okumura ,@salderwe ,@fwinkl ,@stelzer ,@wiedenma ,@mark ,@martindl as watchers

  • added review-pending-level-1 label and removed review-approved label

  • :white_check_mark: CI Result SUCCESS (hash 1eccc042)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    tests :white_check_mark:

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 5442] (remote access info)

  • Looks good to me. L1

  • added review-approved label and removed review-pending-level-1 label

  • Duc Ta mentioned in commit f88d9b30

    mentioned in commit f88d9b30

  • merged

  • Tadej Novak mentioned in merge request !69320 (merged)

    mentioned in merge request !69320 (merged)

  • Oleg Kuprash mentioned in merge request !69365 (merged)

    mentioned in merge request !69365 (merged)

  • Please register or sign in to reply
    Loading