additional step of migration of TrigUpgradeTest: cc code to DecisionHandling and some tests
Content of migration:
- TUT source code to DecisionHandling
- emu_step_* tests to DecisionHandling: as unit tests
- full_menu_cf test to TriggerMenuMT: as configuration only
Related to ATR-21713
Merge request reports
Activity
added Trigger TriggerMenu master review-pending-level-1 labels
❎ CI Result FAILUREAthena AthSimulation AthGeneration AnalysisBase externals ✅ ✅ ✅ ✅ cmake ⭕ ✅ ✅ ✅ make N/A N/A N/A N/A required tests N/A N/A N/A N/A optional tests N/A N/A N/A N/A ❗ Due to problems in externals build or cmake configuration the job is stopped, results are not available on the ATLAS CI monitor✅ 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 17876]- Resolved by Rafal Bielski
As we just discussed in a meeting, this move:
Trigger/TrigValidation/TrigUpgradeTest/share/newJOtest.py → Trigger/TrigSteer/DecisionHandling/share/newJOtest.py
should instead be:
Trigger/TrigValidation/TrigUpgradeTest/share/newJOtest.py → Trigger/TrigValidation/TriggerTest/share/newJOtest.py
and the client(s) should be adapted.
added 179 commits
-
65378eed...ac6c952e - 177 commits from branch
atlas:master
- 088dd709 - Merge remote-tracking branch 'upstream/master' into TUTcleaning
- f73858aa - moved newJOtest.py back to TUT
-
65378eed...ac6c952e - 177 commits from branch
❎ CI Result FAILURE (hash 65378eed)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 17878]❎ CI Result FAILURE (hash f73858aa)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 17881]added 1 commit
- 3749c752 - fix some includes that were lost during merging (?)
❎ CI Result FAILURE (hash eb901abf)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 17894]❎ CI Result FAILURE (hash 3749c752)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 17900]Hi @fpastore,
TriggerMenuMT_full_menu_cf_ctest
is failing with JSON decode issues. Additionally some duplicated classes are found, see http://atlas-computing.web.cern.ch/atlas-computing/links/distDirectory/ci/CIWebArea/nicos_web_areaMRCIbuilds64BC7G8AthenaOpt/NICOS_TestLog_MR-35131-2020-07-27-22-01/ReleaseTests___DuplicateClass-test__DuplicateClass-test__m.html.Tadej (L2)
added review-user-action-required label and removed review-pending-level-1 label