Cleanup ART package: remove the software and only keep CMake
Removing ART software from the ART package. Only keeping the ARTEnvironmentConfig.cmake and CMakeLists.txt in the package to add ART input CVMFS directory to the DATAPATH.
The latest version of ART (1.0.7) is available via "lsetup art"
setupATLAS
asetup <nightly>
lsetup art
Merge request reports
Activity
added Tools master review-pending-level-1 labels
CI Result FAILUREAthena AthSimulation externals cmake make required tests optional tests Full details available at NICOS MR-26461-2019-09-10-21-21
Athena: number of compilation errors 0, warnings 1
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 3144]Hi @tcuhadar,
Trigger tests are failing on the lines callingart.py createpoolfile
which was added following your suggestion, here:
https://gitlab.cern.ch/atlas/athena/blob/master/Trigger/TrigValidation/TrigUpgradeTest/test/exec_TrigUpgradeTest_art_athenaMT.sh#L89
and here:
https://gitlab.cern.ch/atlas/athena/blob/master/Trigger/TrigValidation/TriggerTest/test/exec_athena_art_trigger_validation.sh#L63Cheers,
RafalHi @rbielski,
You need to add the lines
export ATLAS_LOCAL_ROOT_BASE="${ATLAS_LOCAL_ROOT_BASE:-/cvmfs/atlas.cern.ch/repo/ATLASLocalRootBase}" source "${ATLAS_LOCAL_ROOT_BASE}"/user/atlasLocalSetup.sh —quiet lsetup ART
to these tests.
But those also run in nightly ? Nightly script should not have those lines..
Cheers, Tulay
Edited by Tulay Cuhadar Donszelmannmentioned in merge request !26482 (merged)
Relevant fixes for three trigger scripts are done in !26482 (merged), which has to be merged before this one.
CI Result SUCCESSAthena AthSimulation externals cmake make required tests optional tests Full details available at NICOS MR-26461-2019-09-11-15-04
Athena: number of compilation errors 0, warnings 1
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 3171]added review-approved label and removed review-pending-level-1 label
mentioned in commit 10524b38
added sweep:ignore label