Turn down b-tagging logging (again)

It seems like cout messages keep creeping into b-tagging. Whack-a-mole continues...

This merge request affects 1 package:

• PhysicsAnalysis/JetTagging/JetTagPerformanceCalibration/CalibrationDataInterface

This merge request affects 2 files:

• PhysicsAnalysis/JetTagging/JetTagPerformanceCalibration/CalibrationDataInterface/CalibrationDataInterface/CDIReader.h
• PhysicsAnalysis/JetTagging/JetTagPerformanceCalibration/CalibrationDataInterface/Root/CDIReader.cxx

added Analysis BTagging JetEtmiss analysis-review-required main review-pending-level-1 labels

mentioned in merge request !63231 (merged)

added bugfix label

The system determined that CI tests (with names matching "^CITest_SimulationRun(2|3)(FullSim|Hit).*$") are not needed for this code change. They are not run. This is not an indicator to restart the job.

CI Result SUCCESS (hash 6c10cd7b)

	Athena	AnalysisBase	AthAnalysis
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: 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 77400]

Changes look fine from L1 point of view - Approving.

Cheers, Steffen (L1)

added review-approved label

removed review-approved review-pending-level-1 labels

Hey @gouighri do you still need to review this? @skorn why did you remove review-approved here? (I'd thought that the analysis reviewer is supposed to add analysis-review-approved separately)

resolved all threads

added review-approved label

This is obviously a desirable change, but maybe it's worth considering if CDIReader should inherit from AsgMessaging in some follow-up, so that the ATLAS controls on log levels become applicable.

merged

mentioned in commit 540e1cd0

mentioned in merge request !65647 (merged)

mentioned in merge request !65680 (merged)

added sweep:ignore label