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
Move EDProducers from Sequences to Tasks in L1T DQM and Validation configurations #30089
Conversation
The code-checks are being triggered in jenkins. |
type bug-fix |
urgent |
@cmsbuild, please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-30089/15827
|
The tests are being triggered in jenkins. |
A new Pull Request was created by @makortel (Matti Kortelainen) for master. It involves the following packages: DQM/L1TMonitor @benkrikler, @kmaeshima, @schneiml, @andrius-k, @cmsbuild, @rekovic, @jfernan2, @fioriNTU can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1 |
@rekovic |
merge |
PR description:
This PR fixes an "unrunnable schedule" error in many workflows in CMSSW_11_2_X_2020-06-01-2300
The problem was that some EDProducer(s) were in a Task in
L1Reco
step, and put in a Sequence inDQM
step. Before the aforementioned IB these EDProducers were moved from Sequence to Task byconvertToUnscheduled()
, which was removed in #29630, leading to modules associated to a Path to depend on event products produced by producers in EndPath. The proposed fix is to move bunch of non-DQM EDProducers from Seqences to Tasks in L1T DQM and Validation configuration files.Part of #30074, a second attempt of #30081.
PR validation:
Limited matrix and workflows
134.801,134.802,134.803,134.804,134.805,134.806,134.807,134.808,134.809,134.81,134.811,134.812,134.901,134.902,134.903,134.904,134.905,134.906,134.907,134.908,134.909,134.91,134.911,134.912,134.99601,134.99602,134.99603,134.99901,140.55,145.0
run.