TrigAnalysisTest: Don't run AOD checks on RDO_TRIG

The "analysis steps" in TrigAnalysisTest ART tests consist of old (from r21) job options running TrigDecChecker, TrigEDMChecker, TrigEDMAuxChecker in three separate jobs. When adding them initially to the tests, I thought they would work just as well on RDO_TRIG as on AOD. However, it turned out these RecExCommon-based JOs auto-configured offline reconstruction in this case to get to AOD-like objects, so we ended up running offline reconstruction three times in tests where we didn't want to run it at all. Disable the steps for now in the tests ending on RDO_TRIG, until we have new configuration of these checks suitable for Run-3 software. Keep them running in tests ending on AOD.

This should speed up the tests and help a bit with timeouts, ATR-20466

cc @tamartin, @okuprash

This merge request affects 1 package:

• Trigger/TrigValidation/TrigAnalysisTest

This merge request affects 7 files:

• Trigger/TrigValidation/TrigAnalysisTest/test/test_trigAna_HITtoRDOTrig_v1Dev_build.py
• Trigger/TrigValidation/TrigAnalysisTest/test/test_trigAna_HITtoRDOTrig_v1Dev_grid.py
• Trigger/TrigValidation/TrigAnalysisTest/test/test_trigAna_HITtoRDOTrig_v7Primaries_build.py
• Trigger/TrigValidation/TrigAnalysisTest/test/test_trigAna_HITtoRDOTrig_v7Primaries_grid.py
• Trigger/TrigValidation/TrigAnalysisTest/test/test_trigAna_RDOtoRDOTrig_v1Dev_grid.py
• Trigger/TrigValidation/TrigAnalysisTest/test/test_trigAna_RDOtoRDOTrig_v7Primaries_build.py
• Trigger/TrigValidation/TrigAnalysisTest/test/test_trigAna_RDOtoRDOTrig_v7Primaries_grid.py

Adding @okumura ,@jpanduro as watchers

added Trigger master review-pending-level-1 labels

CI Result FAILURE (hash ddd8882a)

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

Please address the required test failures affecting the packages: Trigger_athena_data-test, Trigger_athena_MC-test and RecoPHYSLITEFormatTest_optional-test. (L1)

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

@abnag, what do you mean? These are known failures from yesterday, see the CI status board: https://its.cern.ch/jira/secure/Dashboard.jspa?selectPageId=18130

The Trigger ones are already fixed, just need to rerun the CI. The failed test are actually unrelated to the changes here, but for clarity and confidence let's rerun.

Jenkins please retry a build

This merge request affects 1 package:

• Trigger/TrigValidation/TrigAnalysisTest

This merge request affects 7 files:

• Trigger/TrigValidation/TrigAnalysisTest/test/test_trigAna_HITtoRDOTrig_v1Dev_build.py
• Trigger/TrigValidation/TrigAnalysisTest/test/test_trigAna_HITtoRDOTrig_v1Dev_grid.py
• Trigger/TrigValidation/TrigAnalysisTest/test/test_trigAna_HITtoRDOTrig_v7Primaries_build.py
• Trigger/TrigValidation/TrigAnalysisTest/test/test_trigAna_HITtoRDOTrig_v7Primaries_grid.py
• Trigger/TrigValidation/TrigAnalysisTest/test/test_trigAna_RDOtoRDOTrig_v1Dev_grid.py
• Trigger/TrigValidation/TrigAnalysisTest/test/test_trigAna_RDOtoRDOTrig_v7Primaries_build.py
• Trigger/TrigValidation/TrigAnalysisTest/test/test_trigAna_RDOtoRDOTrig_v7Primaries_grid.py

Adding @okumura ,@jpanduro as watchers

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

CI Result SUCCESS (hash ddd8882a)

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

code changes are clear, approving

Roman(L1)

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

merged

mentioned in commit 62aae8ca

added sweep:ignore label