Ordered critical services creation

This MR is an attempt to reduce number of "force" created services but at the same time allow ordering of that critical one. It is realy for review. Tagging @wlampl @stelzer @jchapman @amete

Hi @tbold

is this still valid?

Cheers, Joerg

Yes. I'll restart on it. I was hoping for feedback earlier but I guess everyone is to busy.

added 8049 commits

• a6154f83...d8365628 - 8048 commits from branch atlas:master
• e137b13e - Do not create all specialised services by default

Compare with previous version

unmarked as a Work In Progress

changed the description

This merge request affects 3 packages:

• Control/AthenaConfiguration
• Control/PerformanceMonitoring/PerfMonVTune
• Simulation/G4Atlas/G4AtlasAlg

Adding @jchapman ,@vpascuzz ,@amete ,@ssnyder as watchers

added Core Simulation master review-pending-level-1 labels

CI Result FAILURE (hash e137b13e)

	Athena	AthSimulation	AthGeneration	AnalysisBase
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 19796]

Please take a look at the failing unit tests. Cheers, L1

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

added NewConfig label

Hi @tbold,

Could you update Simulation/ISF/ISF_Config/test/ISF_MainConfigNew_Test.py too please, the change is the same as the one for Simulation/G4Atlas/G4AtlasAlg/test/G4AtlasAlgConfigNew_Test.py.

Cheers,

John

Jenkins please retry a build

This merge request affects 3 packages:

• Control/AthenaConfiguration
• Control/PerformanceMonitoring/PerfMonVTune
• Simulation/G4Atlas/G4AtlasAlg

Adding @jchapman ,@vpascuzz ,@amete ,@ssnyder as watchers

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

CI Result FAILURE (hash e137b13e)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 1
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 20218]

Hi @tbold, there are many tests failing, could you please check?

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

added 675 commits

• e137b13e...6effd81e - 673 commits from branch atlas:master
• 12aa7531 - Merge remote-tracking branch 'upstream/master' into force-loading-services
• 59dda440 - Removedd MessageSvc and JO SVc from defaults, fixed ISF test

Compare with previous version

This merge request affects 4 packages:

• Control/AthenaConfiguration
• Control/PerformanceMonitoring/PerfMonVTune
• Simulation/G4Atlas/G4AtlasAlg
• Simulation/ISF/ISF_Config

Adding @jchapman ,@vpascuzz ,@amete ,@ssnyder as watchers