Switch off AthSimulation ART-build tests for unsupported Simulation configurations

Should fix ATLASSIM-4019, ATLASSIM-4029.

changed the description

Jenkins please retry a build

This merge request affects 2 packages:

• Simulation/G4Atlas/G4AtlasApps
• Simulation/ISF/ISF_Validation

Adding @jchapman ,@vpascuzz as watchers

added 21.0 Simulation review-pending-level-1 labels

CI Result SUCCESS

	Athena	AthDataQuality	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available at NICOS MR-21074-2019-02-13-21-09
Athena: number of compilation errors 0, warnings 163
AthDataQuality: number of compilation errors 0, warnings 1
AthSimulation: number of compilation errors 0, warnings 6
For experts only: Jenkins output [CI-MERGE-REQUEST 33755]

The changes look ok. There are some errors visible when opening the NICOS page. Since, I do not know how important these errors are, I pass this to L2 shifters to decide if it is fine and if it is needed to report on ATLINFR. I do not see such errors in other MRs.

L1 shifters

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

This merge request affects 2 packages:

• Simulation/G4Atlas/G4AtlasApps
• Simulation/ISF/ISF_Validation

Adding @jchapman ,@vpascuzz as watchers

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

CI Result SUCCESS

	Athena	AthDataQuality	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available at NICOS MR-21074-2019-02-14-15-19
Athena: number of compilation errors 0, warnings 163
AthDataQuality: number of compilation errors 0, warnings 1
AthSimulation: number of compilation errors 0, warnings 6
For experts only: Jenkins output [CI-MERGE-REQUEST 33825]

Fine, approving.

 Pavol [as L2 MR shifter]

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

changed the description

Hi @janders,

Could you merge this one please? It should be very safe as it's just switching off some ART-build tests.

Cheers,

John

mentioned in commit 481b2b0d

merged

Thanks @janders!

No problem, sorry the delay!

added sweep:done label

mentioned in commit 28391286

mentioned in merge request !21345 (merged)

Sweep summary
successful:

• master

(see https://aiatlas154.cern.ch/jenkins/job/auto_git_sweeps/687/console for details)

mentioned in commit e2212714

mentioned in merge request !22121 (merged)

mentioned in commit 7d996a63