-
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
HLT menu 2017 v1.1 #19235
HLT menu 2017 v1.1 #19235
Conversation
A new Pull Request was created by @Martin-Grunewald (Martin Grunewald) for master. It involves the following packages: Configuration/HLT @Martin-Grunewald, @silviodonato, @cmsbuild, @fwyzard, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins. |
-1 Tested at: 19d80d9 You can see the results of the tests here: I found follow errors while testing this PR Failed tests: RelVals AddOn
When I ran the RelVals I found an error in the following worklfows: runTheMatrix-results/10042.0_ZMM_13+ZMM_13TeV_TuneCUETP8M1_2017_GenSimFull+DigiFull_2017+RecoFull_2017+ALCAFull_2017+HARVESTFull_2017/step2_ZMM_13+ZMM_13TeV_TuneCUETP8M1_2017_GenSimFull+DigiFull_2017+RecoFull_2017+ALCAFull_2017+HARVESTFull_2017.log10024.0 step2 runTheMatrix-results/10024.0_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2017_GenSimFull+DigiFull_2017+RecoFull_2017+ALCAFull_2017+HARVESTFull_2017/step2_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2017_GenSimFull+DigiFull_2017+RecoFull_2017+ALCAFull_2017+HARVESTFull_2017.log10824.0 step2 runTheMatrix-results/10824.0_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2018_GenSimFull+DigiFull_2018+RecoFull_2018+ALCAFull_2018+HARVESTFull_2018/step2_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2018_GenSimFull+DigiFull_2018+RecoFull_2018+ALCAFull_2018+HARVESTFull_2018.log
I found errors in the following addon tests: cmsRun /cvmfs/cms-ib.cern.ch/nweek-02476/slc6_amd64_gcc530/cms/cmssw/CMSSW_9_2_X_2017-06-14-2300/src/HLTrigger/Configuration/test/OnLine_HLT_PRef.py realData=True globalTag=@ inputFiles=@ : FAILED - time: date Thu Jun 15 09:25:16 2017-date Thu Jun 15 09:20:42 2017 s - exit: 18688 |
Comparison not run due to runTheMatrix errors (RelVals and Igprof tests were also skipped) |
Pull request #19235 was updated. @Martin-Grunewald, @silviodonato, @cmsbuild, @fwyzard, @davidlange6 can you please check and sign again. |
please test |
The tests are being triggered in jenkins. |
addOn tests should be cured by adding missing OnLine_HLT_PRef.py file. relval tests need an update of GTs for the L1T menu used by the updated HLT menu, |
-1 Tested at: 3cc154a The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: You can see the results of the tests here: I found follow errors while testing this PR Failed tests: RelVals
When I ran the RelVals I found an error in the following worklfows: runTheMatrix-results/10042.0_ZMM_13+ZMM_13TeV_TuneCUETP8M1_2017_GenSimFull+DigiFull_2017+RecoFull_2017+ALCAFull_2017+HARVESTFull_2017/step2_ZMM_13+ZMM_13TeV_TuneCUETP8M1_2017_GenSimFull+DigiFull_2017+RecoFull_2017+ALCAFull_2017+HARVESTFull_2017.log10024.0 step2 runTheMatrix-results/10024.0_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2017_GenSimFull+DigiFull_2017+RecoFull_2017+ALCAFull_2017+HARVESTFull_2017/step2_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2017_GenSimFull+DigiFull_2017+RecoFull_2017+ALCAFull_2017+HARVESTFull_2017.log10824.0 step2 runTheMatrix-results/10824.0_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2018_GenSimFull+DigiFull_2018+RecoFull_2018+ALCAFull_2018+HARVESTFull_2018/step2_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2018_GenSimFull+DigiFull_2018+RecoFull_2018+ALCAFull_2018+HARVESTFull_2018.log The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
Comparison not run due to runTheMatrix errors (RelVals and Igprof tests were also skipped) |
@rekovic reported at AlCa/Db meeting of reproducing locally the problem and indeed ZeroBias not firing. The origin is under investigation , an ETA for the fix in the making. Given the other parallel condition integration fronts @Martin-Grunewald @rekovic @arunhep @kkotov |
+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 requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar |
+1 |
I hope this enters the next release, otherwise it will be difficult to validate on the Strips side @boudoul |
Hi people, |
Nevermind, Vladimir pointed me to 10024.0 . |
@davidlange6 |
CMSSW_9_3_X
|
thanks! |
Looking at this issue and keeping track of it here |
Alca will integrate the fix as soon as available, if, as it seems to date, the origin is the l1 menu |
@deguio can you confirm that this is not part of the hcal validation suite?: https://github.com/cms-sw/cmssw/blob/CMSSW_9_2_X/Validation/GlobalRecHits/src/GlobalRecHitsAnalyzer.cc |
I think that this pull request may be what is causing the RleVal failure for 1325.0. The failure does not happen in 9_2 but does start with the first 9_3. Can anyone confirm or refute the connection? |
Correct - that wf uses HLT:@relval2016 which now redirects to the Fake2 menu containing just a few basic paths. BTW, the filter throwing the exception in question can be configured to print a MessageLogger LogInfo message rather than throw an exception; I suggest doing so, or of course updating the requested pattern "AlCa_EcalPhiSym*" |
L1_ZeroBias fix - PR is #19492 |
Summary of changes in Global TagsRunII data
Upgrade
|
HLT menu 2017 v1.1
Based on 923.
rebased to CMSSW_9_2_X_2017-06-22-2300 (23 June 2017)