IDtrigger hanconfig - first implementation for Run3

First implementation of IDtrig hanconfig for run3

• using new regexes ATR-25596

This merge request affects 1 package:

• DataQuality/DataQualityConfigurations

This merge request affects 1 file:

• DataQuality/DataQualityConfigurations/config/HLT/HLTidtrk/collisions_run.config

Adding @ebergeas as watcher

added DQ master review-pending-level-1 labels

Hi @hartj ,
Good that this works now! Do you have an example webdisplay? (Or is this identical to the version we discussed?)
Thanks!
Elin

CI Result SUCCESS (hash f10b6e27)

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

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

Changes and CI look fine from L1 point of view. Marking as user action required to adress the comment before approving.

Kira (L1)

Hi @ebergeas

See here: https://atlasdqm.cern.ch/webdisplay/test/126/physics_Main/run_360026/run/

Cheers Julie

resolved all threads

removed review-user-action-required label

added review-pending-level-1 label

marked this merge request as draft

Removing review-pending-level-1 since this is a Draft. Re-label when ready -- L1

removed review-pending-level-1 label

resolved all threads

added review-pending-level-1 label

marked this merge request as ready

Hi @cvarni , Sorry I got mixed up between merge requests - this one is NOT draft. Could you please review it? Cheers Julie

changed target branch from master to 22.0

Sorry this was supposed to go to 22.0

Changed the target branch now

This merge request affects 1 package:

• DataQuality/DataQualityConfigurations

This merge request affects 1 file:

• DataQuality/DataQualityConfigurations/config/HLT/HLTidtrk/collisions_run.config

Adding @ebergeas as watcher

added 22.0 label

added review-approved-point1 label

removed master label

CI Result SUCCESS (hash f10b6e27)

	Athena	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
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 53541]

added review-approved-tier0 label

added urgent label

Just checking: Are those deletions on purpose? There is one commented line https://gitlab.cern.ch/atlas/athena/-/blob/f10b6e274fdf59d9482b124dd874b686e93b5658/DataQuality/DataQualityConfigurations/config/HLT/HLTidtrk/collisions_run.config#L47 (sorry, cannot comment on the file directly, sice the diff is too large) Otherwise looks fine from an L1 point of view. L1

removed review-pending-level-1 label

added review-pending-level-2 review-user-action-required labels

added 1 commit

• d489cdf0 - remove commented line

Compare with previous version

This merge request affects 1 package:

• DataQuality/DataQualityConfigurations

This merge request affects 1 file:

• DataQuality/DataQualityConfigurations/config/HLT/HLTidtrk/collisions_run.config

Adding @ebergeas as watcher

added review-pending-level-1 label and removed review-approved-point1 review-approved-tier0 review-pending-level-2 review-user-action-required labels

The deletions are on purpose - with the new hanconfig regexes we have a much simpler config file.

The commented line is now removed. Since this is the only change do we need to wait for another round of the CI??

removed review-pending-level-1 label

added review-pending-level-2 label

added review-approved-tier0 label

added review-approved-point1 label

resolved all threads

All good. Karolos (L2)

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

CI Result SUCCESS (hash d489cdf0)

	Athena	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
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 53627]

mentioned in commit e0706fe2

merged