Move or add trhead unsafety reasons to lines of ATLAS_NOT_THREAD_SAFE

• Move or add the reasons of trhead unsafety to the lines of ATLAS_NOT_THREAD_SAFE or ATLAS_NO_CHECK_FILE_THREAD_SAFETY for easier understanding using LXR and git grep.
  • https://acode-browser1.usatlas.bnl.gov/lxr/search?%21v=head&_filestring=InnerDetector&_string=ATLAS_NOT_THREAD_SAFE
  • https://acode-browser1.usatlas.bnl.gov/lxr/search?%21v=head&_filestring=InnerDetector&_string=ATLAS_NO_CHECK_FILE_THREAD
• Remove ATLAS_NOT_THREAD_SAFE from TRT_BSErrContainerCnv_p1::transToPers and TRT_BSErrContainerCnv_p2::transToPers.
• No result changes are expected.
• https://its.cern.ch/jira/browse/ATLASRECTS-5560

changed title from Move trhead unsafety reasons to lines of ATLAS_NOT_THREAD_SAFE to Move or add trhead unsafety reasons to lines of ATLAS_NOT_THREAD_SAFE

changed the description

WARNING: big files (>100K) are found in the changeset

152K in file InnerDetector/InDetCalibAlgs/SCT_CalibAlgs/src/SCTCalib.cxx

This merge request affects 17 packages:

• InnerDetector/InDetCalibAlgs/SCT_CalibAlgs
• InnerDetector/InDetCalibTools/TRT_CalibTools
• InnerDetector/InDetConditions/InDetBeamSpotService
• InnerDetector/InDetConditions/InDetByteStreamErrorsAthenaPool
• InnerDetector/InDetDetDescr/SCT_GeoModel
• InnerDetector/InDetDetDescr/TRT_ReadoutGeometry
• InnerDetector/InDetDigitization/TRT_Digitization
• InnerDetector/InDetEventCnv/InDetEventTPCnv
• InnerDetector/InDetEventCnv/TRT_RawDataByteStreamCnv
• InnerDetector/InDetMonitoring/InDetDiMuonMonitoring
• InnerDetector/InDetRawAlgs/InDetOverlay
• InnerDetector/InDetRecAlgs/InDetLowBetaFinder
• InnerDetector/InDetRecTools/InDetAmbiTrackSelectionTool
• InnerDetector/InDetRecTools/InDetVKalVxInJetTool
• InnerDetector/InDetRecTools/TRT_TrackSegmentsTool_xk
• InnerDetector/InDetTrigRecAlgs/InDetTrigAmbiguitySolver
• InnerDetector/InDetValidation/InDetPhysValMonitoring

Adding @goetz ,@cgrefe ,@amorley ,@sutt ,@nagano ,@tkharlam ,@sroe ,@csuster ,@jchapman ,@bernius ,@calfayan ,@ahaas ,@oda ,@tadej ,@gavrilen as watchers

added DQ Digitization InnerDetector JetEtmiss Overlay Trigger master review-pending-level-1 labels

CI Result SUCCESS (hash 72fe48b4)

	Athena	AthSimulation	AnalysisBase	AthGeneration
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
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 16420]

The changes look ok. Adding review-approved label.

L1 shifter

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

merged

mentioned in commit 78fad04c

added sweep:ignore label