Added method to collect data dependencies of CA

This the initial implementation of a method that allows answering questions about the IO of the CA. The intention is that it will give an early warning that a CA is incomplete. Specifically, in HLT, we expect that CAs representing a reco of a single step will only require extra dependencies that are explicitly known. With this functionality, we will be able to check it. (@bwynne will enjoy it). Also tagging @fwinkl FI as it is all based on his work in Gaudi data handles. Tagging @wlampl FI

One caveat is that private tools that are not assigned are not checked. I think we could do that if needed. I.e. we can always instantiate them and pick necessary properties.

This merge request affects 1 package:

• Control/AthenaConfiguration

This merge request affects 2 files:

• Control/AthenaConfiguration/python/ComponentAccumulator.py
• Control/AthenaConfiguration/python/ComponentAccumulatorTest.py

Adding @ssnyder as watcher

added Core master review-pending-level-1 labels

Looks great! Thanks Tomasz.

added 92 commits

• 3a2a5fa7...dfd10443 - 91 commits from branch atlas:master
• a31d5de5 - Fixed conflict

Compare with previous version

This merge request affects 1 package:

• Control/AthenaConfiguration

This merge request affects 2 files:

• Control/AthenaConfiguration/python/ComponentAccumulator.py
• Control/AthenaConfiguration/python/ComponentAccumulatorTest.py

Adding @ssnyder as watcher

CI Result FAILURE (hash 3a2a5fa7)

	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 29176]

added NewConfig label

added 1 commit

• 54b10eb0 - Run IO test only in Athena project

Compare with previous version

This merge request affects 1 package:

• Control/AthenaConfiguration

This merge request affects 2 files:

• Control/AthenaConfiguration/python/ComponentAccumulator.py
• Control/AthenaConfiguration/python/ComponentAccumulatorTest.py

Adding @ssnyder as watcher

CI Result FAILURE (hash a31d5de5)

	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 29193]

CI Result SUCCESS (hash 54b10eb0)

	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 29196]

Setting review-user-action-required until open discussions are resolved.

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

added 1 commit

• e9cc0458 - Post review fixes

Compare with previous version

This merge request affects 1 package:

• Control/AthenaConfiguration

This merge request affects 4 files:

• Control/AthenaConfiguration/CMakeLists.txt
• Control/AthenaConfiguration/python/ComponentAccumulator.py
• Control/AthenaConfiguration/python/ComponentAccumulatorTest.py
• Control/AthenaConfiguration/python/ComponentAccumulatorTestReco.py

Adding @ssnyder as watcher

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

CI Result SUCCESS (hash e9cc0458)

	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 29244]

Hi,

this looks fine from L1 side. Just to be on the safe side: can the remaining discussion be resolved?

Thanks, Volker (L1)

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

resolved all threads

added review-approved label and removed review-user-action-required label

merged

mentioned in commit f0bc2fb1

added sweep:ignore label