LArRawDataContByteStreamTool migrated to retrieving CaloDetDescrManager from the Condition Store

This is part of the ongoing migration of all clients of the CaloDetDescrManager

This merge request affects 1 package:

• LArCalorimeter/LArCnv/LArByteStream

This merge request affects 2 files:

• LArCalorimeter/LArCnv/LArByteStream/LArByteStream/LArRawDataContByteStreamTool.h
• LArCalorimeter/LArCnv/LArByteStream/src/LArRawDataContByteStreamTool.cxx

Adding @pavol as watcher

added JetEtmiss LAr master review-pending-level-1 labels

CI Result FAILURE (hash d9f44d16)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional 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
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 45319]

added review-user-action-required label and removed review-pending-level-1 label

added 142 commits

• d9f44d16...160d8212 - 141 commits from branch atlas:master
• 0c6ef128 - LArRawDataContByteStreamTool migrated to retrieving CaloDetDescrManager from the Condition Store

Compare with previous version

This merge request affects 1 package:

• LArCalorimeter/LArCnv/LArByteStream

This merge request affects 5 files:

• LArCalorimeter/LArCnv/LArByteStream/LArByteStream/LArLATOMEHeaderContByteStreamCnv.h
• LArCalorimeter/LArCnv/LArByteStream/LArByteStream/LArRawDataContByteStreamTool.h
• LArCalorimeter/LArCnv/LArByteStream/LArByteStream/LArRawScContByteStreamCnv.h
• LArCalorimeter/LArCnv/LArByteStream/python/LArByteStreamConfig.py
• LArCalorimeter/LArCnv/LArByteStream/src/LArRawDataContByteStreamTool.cxx

Adding @pavol as watcher

added review-pending-level-1 label and removed review-user-action-required label

CI Result SUCCESS (hash 0c6ef128)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional 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
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 45453]

The changes look fine and there are no CI issues. Approving.

Jason (L1)

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

mentioned in commit 9dcb4f34

merged

added sweep:ignore label