CaloAlignCondAlg updates
- Allow empty keys for read condition handles.
When both of keys are empty, then the algorithm builds an unaligned Calo DD Manager and writes it into Condition Store with infinite validity. This is needed for some unit tests (see !47007 (merged))
- Downgrade
CaloAlignCondAlg
from reentrant to regular algorithm
Hopefully a temporary measure to quickly fix random failures in some tests (see ATR-24282).
In the long run the threading issues in CaloAlignCondAlg
need to be understood
and fixed. After that we can turn it into a reentrant algorithm again
Merge request reports
Activity
This merge request affects 2 packages:
- Calorimeter/CaloAlignment/CaloAlignmentAlgs
- LArCalorimeter/LArGeoModel/LArGeoAlgsNV
Affected files list will not be printed in this case
Adding @pavol as watcher
added Calorimeter JetEtmiss LAr master review-pending-level-1 labels
mentioned in merge request !47007 (merged)
CI Result SUCCESS (hash 6f0290a6)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 40042]added review-approved label and removed review-pending-level-1 label
Hi @fwinkl: You would do me a favor by merging this one quickly!
Thanks, Walter
mentioned in commit 1e41df77
added sweep:ignore label