Migrate GeoModel.Run to be an enum

Migrate GeoModel.Run to be an enum.

added NewConfig full-unit-tests labels

This merge request affects 22 packages. Since this is a long list, it will not be printed.

Adding @jchapman ,@cgrefe ,@joheinri ,@duperrin ,@stavrop ,@battagl ,@rosati ,@solodkov ,@jojungge ,@bcarlson ,@stsuno ,@egodden ,@dguest ,@damazio ,@lidiaz ,@goetz ,@amorley ,@wleight ,@pavol ,@sroe ,@harkusha ,@calfayan ,@lshan ,@ssnyder ,@rbianchi as watchers

added Analysis BTagging Core DQ Derivation Digitization Egamma Geometry InnerDetector JetEtmiss LAr MuonSpectrometer Reconstruction Simulation Tile Tracking master review-pending-level-1 labels

Was a bit to fast here. Will push again in a minute.

added 1 commit

• 060c7dc6 - Migrate GeoModel.Run to be an enum

Compare with previous version

This merge request affects 22 packages. Since this is a long list, it will not be printed.

Adding @jchapman ,@cgrefe ,@joheinri ,@duperrin ,@stavrop ,@battagl ,@rosati ,@solodkov ,@jojungge ,@bcarlson ,@stsuno ,@egodden ,@dguest ,@damazio ,@lidiaz ,@goetz ,@amorley ,@wleight ,@pavol ,@sroe ,@harkusha ,@calfayan ,@lshan ,@ssnyder ,@rbianchi as watchers

CI Result FAILURE (hash 060c7dc6)

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

CaloClusterCorrection_config_test_ctest is unrelated, but EventSelectorAthenaPool_CondProxyProviderConfig_test_ctest and AthAnalysis ones are.

removed review-pending-level-1 label

added review-user-action-required label

added 25 commits

• 060c7dc6...b531b586 - 24 commits from branch atlas:master
• 87683868 - Migrate GeoModel.Run to be an enum

Compare with previous version

This merge request affects 24 packages. Since this is a long list, it will not be printed.

Adding @jchapman ,@cgrefe ,@joheinri ,@harkusha ,@duperrin ,@stavrop ,@battagl ,@rosati ,@solodkov ,@jojungge ,@bcarlson ,@stsuno ,@egodden ,@dguest ,@damazio ,@lidiaz ,@goetz ,@amorley ,@wleight ,@pavol ,@sroe ,@mnowak ,@calfayan ,@lshan ,@tadej ,@ssnyder ,@rbianchi as watchers

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

CI Result FAILURE (hash 87683868)

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

Test failures unrelated. Marking urgent to avoid conflicts.

added urgent label

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

Changes seem fine to me. CaloClusterCorrection_config_test_ctest and TrigConfData_TestDoubleMenuLoad_ctest failures seem unrelated to them. Approving. -L1

mentioned in commit 20c93a63

merged

mentioned in merge request !50300 (merged)

added sweep:ignore label