Additional MT protection
- There can be multiple thinning services in RAWtoALL, change the mutex here to be static to cover them all.
- The thinning service and monitoring both make use of the same underlying (singleton) thread-hostile legacy trigger navigation tool. !40884 (merged) reduced the number of issues, but did not eliminate them.
This MR serialises all legacy HLT monitoring and thinning.
It reduces the frequency of MT-crashes, but still does not eliminate them. A x3 follow up will still be needed.
Merge request reports
Activity
This merge request affects 2 packages:
- Control/AthenaMonitoring
- Trigger/TrigEvent/TrigNavTools
Affected files list will not be printed in this case
Adding @ssnyder as watcher
added Core DQ Trigger master review-pending-level-1 labels
This merge request affects 2 packages:
- Control/AthenaMonitoring
- Trigger/TrigEvent/TrigNavTools
Affected files list will not be printed in this case
Adding @ssnyder as watcher
CI Result SUCCESS (hash 5dcb6e1c)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis 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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 29083] CI Result SUCCESS (hash 1ff911f2)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis 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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 29081]added review-approved label and removed review-pending-level-1 label
mentioned in commit 73b7d0bd
added sweep:ignore label