update HLT selection for HI mode in sistrip and l1tstage2 online-DQM clients

[missirol]

PR description:

This PR applies simple fixes to the HLT selections used in the sistrip and l1tstage2 online-DQM clients, based on the latest version of the HLT menu for the 2023 HIon run (see here).

• sistrip: triggers named HLT_HICentralityVeto* in the HIon menu have been replaced by triggers named HLT_HIMinimumBias*, so this client is updated accordingly.
• l1tstage2: the HLT selection for HI mode is fixed by adding HLT_HIPhysics_v* (needed for DQM involving the inputs of all the different uGT boards sent only in "L1T fat events"); this follows what is already being done in the l1tstage2emulator client.

Minimal code-style updates are also included.

PR validation:

None.

If this PR is a backport, please specify the original PR and why you need to backport that PR. If this PR will be backported, please specify to which release cycle the backport is meant for:

CMSSW_13_2_X

[missirol]

Just highlighting the only 2 significant changes in this PR.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-42803/36916

• This PR adds an extra 24KB to repository

[cmsbuild]

A new Pull Request was created by @missirol (Marino Missiroli) for master.

It involves the following packages:

• DQM/Integration (dqm)

@nothingface0, @emanueleusai, @cmsbuild, @pmandrik, @syuvivida, @tjavaid, @micsucmed, @rvenditti can you please review it and eventually sign? Thanks.
@threus, @batinkov, @francescobrivio this is something you requested to watch as well.
@rappoccio, @antoniovilela, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-42803/36925

• This PR adds an extra 24KB to repository

[cmsbuild]

Pull request #42803 was updated. @nothingface0, @emanueleusai, @cmsbuild, @pmandrik, @syuvivida, @tjavaid, @micsucmed, @rvenditti can you please check and sign again.

[mmusich]

@cmsbuild, please test

• let's start testing it (even though the relevant paths will not be exercised in unit tests)

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-45caa6/34809/summary.html
COMMIT: 170b35b
CMSSW: CMSSW_13_3_X_2023-09-18-2300/el8_amd64_gcc11
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/42803/34809/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• You potentially removed 3 lines from the logs
• Reco comparison results: 11 differences found in the comparisons
• DQMHistoTests: Total files compared: 50
• DQMHistoTests: Total histograms compared: 3358044
• DQMHistoTests: Total failures: 6
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3358016
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 49 files compared)
• Checked 214 log files, 167 edm output root files, 50 DQM output files
• TriggerResults: no differences found

[emanueleusai]

+1

• test at p5 ok

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @rappoccio, @antoniovilela, @sextonkennedy (and backports should be raised in the release meeting by the corresponding L2)

[rappoccio]

+1