-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Modernize DQM/BeamMonitor/plugins/TKStatus
plugin
#37214
Modernize DQM/BeamMonitor/plugins/TKStatus
plugin
#37214
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37214/28814
|
A new Pull Request was created by @francescobrivio for master. It involves the following packages:
@emanueleusai, @ahmad3213, @cmsbuild, @jfernan2, @pmandrik, @pbo0, @rvenditti can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@francescobrivio cmssw/DQM/Integration/python/config/unittestinputsource_cfi.py Lines 119 to 124 in 523b360
precisely to catch this sort of issues upstream and not when the client is run online at P5 :) |
7b27e36
to
fab8b2b
Compare
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37214/28825
|
Pull request #37214 was updated. @emanueleusai, @ahmad3213, @cmsbuild, @jfernan2, @pmandrik, @pbo0, @rvenditti can you please check and sign again. |
DQM/BeamMonitor/plugins/TKStatus
pluginDQM/BeamMonitor/plugins/TKStatus
plugin
@cmsbuild please test |
@cms-sw/dqm-l2 do we also need a 12_2 backport? |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c7665a/23067/summary.html Comparison SummarySummary:
|
I opened a 12_2_X backport in #37224 |
+1 |
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, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
In the same spirit of #29198, #35782 and #37084, this PR modernizes the
DQM/BeamMonitor/plugins/TKStatus
plugin to useDCSRecord
token in case the DCS information from SCAL is not available (i.e. from Run3 onwards).Changes:
TKStatus.cc
andTKStatus.h
DCSRecord
consumption (and a logic to choose between it andDCSstatus
)process.onlineMetaDataDigis
unpacker in the only DQM client running in production that uses TkStatusprocess.onlineMetaDataDigis
unpacker to a bunch of configs underDQM/BeamMonitor/test
that were importing and using TkStatusPR validation:
Code compiles.
Tested by running on run
348490
:which has only Pixels ON.
And by running on run
348587
:which has both Pixel and Strips ON.
Backport
A backport to 12_3_X is provided in #37223
A backport to 12_2_X is provided in #37224