Add check of Tracker HV in BeamSpot DQM clients

[francescobrivio]

PR description:

In this PR a few improvements to the BeamSpot DQM clients are made:

• A check on the Tracker HV (both pixels and strips) is added in order not to run on events which are by construction not useful for the BeamSpot determination
  • This will alleviate the stress on the CondDB unfrastructure by reducing the number of "fake-failing" BeamSpot logs that are uploaded to the DB and that are monitored by the DB Experts on call (FYI @cms-sw/db-l2)
  • The HV check in the beamhlt client requires a change in the event content of the streamDQMOnlineBeamspot to include the raw data from FED 1022 via a new hltFEDSelectorOnlineMetaData
    This is being implemented by @missirol as described in JIRA-CMSHLT-2464
  • No additional changes are needed for the HV check in the beam client as the full raw data information is already available in the standard streamDQM event content
• An explicit printout of the Global Tag used by the client is added to the client logs
• A new unitteststreamerinputsource_cfi.py config file has been added for the specific cases where the unitTests are running on streamer files
  • Currently only the beamhlt DQM client is using this feature, but more clients could be adapted in the future
  • The new config allows to avoid querying DAS uselessly since the input streamer files are read from cms-data/DQM-Integration
• A typo in the LogInfo printout of RecoVertex/BeamSpotProducer/plugins/OnlineBeamSpotESProducer.cc has been fixed

PR validation:

Code compiles plus successfully run:

scram b runtests_TestDQMOnlineClient-beam_dqm_sourceclient 
scram b runtests_TestDQMOnlineClient-beamhlt_dqm_sourceclient

Backport:

Not a backport but 12_5_X and 12_4_X backports will be opened soon.

FYI @gennai @dzuolo @mmusich @ggovi

[francescobrivio]

type performance-improvement

[francescobrivio]

@cms-sw/dqm-l2 please advise if you agree with the unitteststreamerinputsource_cfi.py name or if you prefer a different nomenclature.
Also, please let me know if you agree with the default options that are registered in this new config file.

I'll wait for your feedback on these two points before opening the backports, but keep in mind it would be nice to have this merged and backported in the new 12_4_X for when the collision data-taking restarts.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39347/32049

• This PR adds an extra 24KB to repository

[cmsbuild]

A new Pull Request was created by @francescobrivio for master.

It involves the following packages:

• DQM/Integration (dqm)
• RecoVertex/BeamSpotProducer (reconstruction, alca)

@malbouis, @yuanchao, @micsucmed, @emanueleusai, @ahmad3213, @tvami, @cmsbuild, @ChrisMisan, @jfernan2, @clacaputo, @pmandrik, @saumyaphor4252, @francescobrivio, @mandrenguyen, @rvenditti can you please review it and eventually sign? Thanks.
@VourMa, @battibass, @mtosi, @GiacomoSguazzoni, @JanFSchulte, @tocheng, @VinInn, @rovere, @mmusich, @threus, @dgulhan, @batinkov this is something you requested to watch as well.
@perrotta, @dpiparo, @rappoccio you are the release manager for this.

cms-bot commands are listed here

[francescobrivio]

@cmsbuild please test

[francescobrivio]

@cms-sw/reconstruction-l2 just FYI the reco change here is only a typo in a LogInfo printout :D

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-66b1df/27427/summary.html
COMMIT: 55dd8ee
CMSSW: CMSSW_12_6_X_2022-09-07-2300/el8_amd64_gcc10
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/39347/27427/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• No significant changes to the logs found
• Reco comparison results: 4 differences found in the comparisons
• DQMHistoTests: Total files compared: 51
• DQMHistoTests: Total histograms compared: 3618326
• DQMHistoTests: Total failures: 5
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3618299
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 50 files compared)
• Checked 212 log files, 49 edm output root files, 51 DQM output files
• TriggerResults: no differences found

[emanueleusai]

+1

[mandrenguyen]

+reconstruction
reco only triggered b/c a typo was fixed in a LogInfo

[tvami]

+alca

[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. @perrotta, @dpiparo, @rappoccio (and backports should be raised in the release meeting by the corresponding L2)

[perrotta]

+1

[syuvivida]

+1