ECAL online DQM timing conditions tag override - 132x - do not merge

[thomreis]

PR description:

This PR is intended for the online DQM during the HI run 2023. Do not merge it in official 13_2_X.

The ECAL timing calibration constants and timing offset constants in the GT will be overridden by tags containing conditions for the CC timing. This is needed because the HLT GT used in the online DQM configuration contains the conditions for the ratio timing method running at the HLT.

PR validation:

Run the ecal_dqm_sourceclient-live_cfg.py configuration.

[cmsbuild]

A new Pull Request was created by @thomreis (Thomas Reis) for CMSSW_13_2_X.

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.
@batinkov, @rchatter, @argiro, @thomreis, @wang0jin, @threus, @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

[thomreis]

please test

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b993ad/34891/summary.html
COMMIT: ad270a4
CMSSW: CMSSW_13_2_X_2023-09-25-1100/el8_amd64_gcc11
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/42855/34891/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: 7 differences found in the comparisons
• DQMHistoTests: Total files compared: 48
• DQMHistoTests: Total histograms compared: 3198243
• DQMHistoTests: Total failures: 9
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3198212
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 47 files compared)
• Checked 207 log files, 159 edm output root files, 48 DQM output files
• TriggerResults: no differences found

[tjavaid]

+1

[cmsbuild]

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

[tjavaid]

hold

• a temporary fix, not intended to be merged it in official 13_2_X.

[cmsbuild]

Pull request has been put on hold by @tjavaid
They need to issue an unhold command to remove the hold state or L1 can unhold it for all

[tjavaid]

@thomreis , perhaps you would like to close this PR since #42933 has already been there and deployed online?

[thomreis]

Hi @tjavaid yes we are closing this PR.
We were not notified that it had been deployed yesterday already in fact.