-
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
Updating DT Online-DQM to work with slice test data #26651
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-26651/9612
|
A new Pull Request was created by @battibass (Carlo Battilana) for master. It involves the following packages: DQM/DTMonitorModule @andrius-k, @kmaeshima, @schneiml, @cmsbuild, @jfernan2, @fioriNTU can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+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. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
This PR includes recent changes to the
DTDigiTask
module alongside with the necessary DQM configuration file changes to prepare the DT DQM to process data from the DT slice test.Though the DQM monitoring for the DT slice test will initially be performed only exploiting a "private" DT-DQM deployment @ P5, a PR is anyhow prepared to keep the DQM code in the release as much "up to date" as possible.
The PR only affects Online-DQM code, and the updates are not supposed to change the behaviour of the DT-DQM, if it is run on data other than the slice test one. So no changes are expected.
PR validation:
The PR has been tested within a private deployment of the DT Online-DQM.