Update HLT menu (V4) [81X] #15875

silviodonato · 2016-09-16T12:14:48Z

This PR updates the HLT menus to the most updated version (V4).
HLT master table: /dev/CMSSW_8_0_0/HLT/V536
GRun table: /dev/CMSSW_8_0_0/GRun/V184

All working paths that were removed from GRun have been re-inserted, in order to have them available in the next MC production as requested by XEB.

80X version: #15873

silviodonato · 2016-09-16T12:15:01Z

please test

cmsbuild · 2016-09-16T12:15:19Z

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/15191/console

cmsbuild · 2016-09-16T12:15:20Z

A new Pull Request was created by @silviodonato (Silvio Donato) for CMSSW_8_1_X.

It involves the following packages:

Configuration/HLT
HLTrigger/Configuration

@perrotta, @cmsbuild, @silviodonato, @Martin-Grunewald, @fwyzard, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @Martin-Grunewald, @geoff-smith, @jalimena this is something you requested to watch as well.
@slava77, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

mmusich · 2016-09-16T13:44:16Z

Configuration/HLT/python/autoCondHLT.py

    '25ns10e33_v2' : ( ','.join( [ 'L1Menu_Collisions2016_v4_xml'                            ,l1tMenuRecord,connectionString,l1tMenuLabel, "2016-05-24 14:55:00.000"] ), ),
    'HIon'         : ( ','.join( [ 'L1Menu_CollisionsHeavyIons2015_v5_uGT_xml'               ,l1tMenuRecord,connectionString,l1tMenuLabel, "2016-03-04 15:00:00.000"] ), ),
-    'PIon'         : ( ','.join( [ 'L1Menu_Collisions2016_v6_xml'                            ,l1tMenuRecord,connectionString,l1tMenuLabel, "2016-07-19 10:00:00.000"] ), ),
+    'PIon'         : ( ','.join( [ 'L1Menu_Collisions2016_v6r5_ugt_1board_xml'               ,l1tMenuRecord,connectionString,l1tMenuLabel, "2016-08-23 20:00:00.000"] ), ),


@silviodonato, we are updating in parallel the content of the L1 menu in Global Tag. It would be nice to avoid customization once integrated @franzoni

Marco,, these are the TSG internal GTs where we do that all the time....

@Martin-Grunewald so are we expecting this PR to fail the tests?

It depends: currently the @RelVal still points to the previous menu.
We should probably change it to point to the new frozen menu, and then the relvals
which of course use official GTs will fail until those GTs are updated with the new L1 menu.
How do we proceed? You make new GTs and a PR whose commit we add to this one as in the past?

I am updating the GTs in this very moment. I'll notify you once I have the commit done, so that you can cherry-pick it and change the @relval menu.

For 80 and 81? Given that MC production with the new HLT menu is with 80, and 81 should not lag behind 81....

I guess your last statement, not to be circular, should read:

81 should not lag behind 80

....

so both, but 81x first.

In case you are willing to do the change #15879 might be interesting for you

Martin-Grunewald · 2016-09-16T15:30:50Z

please test

cmsbuild · 2016-09-16T15:31:09Z

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/15209/console

cmsbuild · 2016-09-16T16:45:54Z

-1

Tested at: 070a79f

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
0863dcd
bff4852
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15875/15191/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15875/15191/git-merge-result

You can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15875/15191/summary.html

I found follow errors while testing this PR

Failed tests: RelVals AddOn

RelVals:

When I ran the RelVals I found an error in the following worklfows:
136.731 step2

runTheMatrix-results/136.731_RunSinglePh2016B+RunSinglePh2016B+HLTDR2_2016+RECODR2_2016reHLT_skimSinglePh_HIPM+HARVESTDR2/step2_RunSinglePh2016B+RunSinglePh2016B+HLTDR2_2016+RECODR2_2016reHLT_skimSinglePh_HIPM+HARVESTDR2.log

1306.0 step2

runTheMatrix-results/1306.0_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15/step2_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15.log

135.4 step1

runTheMatrix-results/135.4_ZEE_13+ZEEFS_13+HARVESTUP15FS+MINIAODMCUP15FS/step1_ZEE_13+ZEEFS_13+HARVESTUP15FS+MINIAODMCUP15FS.log

1330.0 step2

runTheMatrix-results/1330.0_ZMM_13+ZMM_13+DIGIUP15+RECOUP15+HARVESTUP15/step2_ZMM_13+ZMM_13+DIGIUP15+RECOUP15+HARVESTUP15.log

25202.0 step2

runTheMatrix-results/25202.0_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15_PU25/step2_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15_PU25.log

AddOn:

I found errors in the following addon tests:

cmsDriver.py TTbar_13TeV_TuneCUETP8M1_cfi --conditions auto:run2_mc --fast -n 100 --eventcontent AODSIM,DQM --relval 100000,1000 -s GEN,SIM,RECOBEFMIX,DIGI:pdigi_valid,L1,DIGI2RAW,L1Reco,RECO,EI,HLT:@relval2016,VALIDATION --customise=HLTrigger/Configuration/CustomConfigs.L1THLT --datatier GEN-SIM-DIGI-RECO,DQMIO --beamspot NominalCollision2015 --era Run2_2016 --magField 38T_PostLS1 : FAILED - time: date Fri Sep 16 18:27:24 2016-date Fri Sep 16 18:09:59 2016 s - exit: 16640

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
0863dcd
bff4852
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15875/15191/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15875/15191/git-merge-result

Martin-Grunewald · 2016-09-16T16:48:33Z

OK, here we point to the development menus which need the new L1T menu in the GTs.

mmusich · 2016-09-16T16:58:23Z

please cherry pick from #15879
I cannot test it (for the same reason but in the opposite direction)

cmsbuild · 2016-09-16T18:05:36Z

-1

Tested at: 070a79f

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
0863dcd
bff4852
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15875/15209/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15875/15209/git-merge-result

You can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15875/15209/summary.html

I found follow errors while testing this PR

Failed tests: RelVals AddOn

RelVals:

When I ran the RelVals I found an error in the following worklfows:
136.731 step2

runTheMatrix-results/136.731_RunSinglePh2016B+RunSinglePh2016B+HLTDR2_2016+RECODR2_2016reHLT_skimSinglePh_HIPM+HARVESTDR2/step2_RunSinglePh2016B+RunSinglePh2016B+HLTDR2_2016+RECODR2_2016reHLT_skimSinglePh_HIPM+HARVESTDR2.log

1306.0 step2

runTheMatrix-results/1306.0_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15/step2_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15.log

135.4 step1

runTheMatrix-results/135.4_ZEE_13+ZEEFS_13+HARVESTUP15FS+MINIAODMCUP15FS/step1_ZEE_13+ZEEFS_13+HARVESTUP15FS+MINIAODMCUP15FS.log

1330.0 step2

runTheMatrix-results/1330.0_ZMM_13+ZMM_13+DIGIUP15+RECOUP15+HARVESTUP15/step2_ZMM_13+ZMM_13+DIGIUP15+RECOUP15+HARVESTUP15.log

25202.0 step2

runTheMatrix-results/25202.0_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15_PU25/step2_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15_PU25.log

AddOn:

I found errors in the following addon tests:

cmsDriver.py TTbar_13TeV_TuneCUETP8M1_cfi --conditions auto:run2_mc --fast -n 100 --eventcontent AODSIM,DQM --relval 100000,1000 -s GEN,SIM,RECOBEFMIX,DIGI:pdigi_valid,L1,DIGI2RAW,L1Reco,RECO,EI,HLT:@relval2016,VALIDATION --customise=HLTrigger/Configuration/CustomConfigs.L1THLT --datatier GEN-SIM-DIGI-RECO,DQMIO --beamspot NominalCollision2015 --era Run2_2016 --magField 38T_PostLS1 : FAILED - time: date Fri Sep 16 19:44:19 2016-date Fri Sep 16 19:21:24 2016 s - exit: 16640

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
0863dcd
bff4852
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15875/15209/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15875/15209/git-merge-result

cmsbuild · 2016-09-17T08:04:55Z

Pull request #15875 was updated. @perrotta, @cmsbuild, @ghellwig, @silviodonato, @cerminar, @Martin-Grunewald, @franzoni, @fwyzard, @mmusich, @davidlange6 can you please check and sign again.

cmsbuild · 2016-09-18T07:33:18Z

Pull request #15875 was updated. @perrotta, @cmsbuild, @ghellwig, @silviodonato, @cerminar, @Martin-Grunewald, @franzoni, @fwyzard, @mmusich, @davidlange6 can you please check and sign again.

Martin-Grunewald · 2016-09-18T07:33:38Z

please test

cmsbuild · 2016-09-18T07:33:57Z

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/15249/console

Martin-Grunewald · 2016-09-18T07:34:13Z

Rebased to CMSSW_8_1_X_2016-09-18-0000 and including revised #15879

cmsbuild · 2016-09-18T10:53:14Z

+1
Tested at: b30818f
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15875/15249/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
918603f
10012c1
8201a5a
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15875/15249/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15875/15249/git-merge-result

Martin-Grunewald · 2016-09-18T11:37:55Z

+1

Martin-Grunewald · 2016-09-18T11:38:27Z

@mmusich @cerminar
please sign for AlCa

cmsbuild · 2016-09-18T11:45:56Z

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15875/15249/summary.html

mmusich · 2016-09-18T14:22:33Z

+1

cmsbuild · 2016-09-18T14:22:54Z

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_1_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @smuzaffar

davidlange6 · 2016-09-19T06:06:46Z

+1

cmsbuild added this to the Next CMSSW_8_1_X milestone Sep 16, 2016

cmsbuild added comparison-pending hlt-pending orp-pending pending-signatures labels Sep 16, 2016

cmsbuild added the tests-pending label Sep 16, 2016

cmsbuild added tests-started and removed tests-pending labels Sep 16, 2016

silviodonato mentioned this pull request Sep 16, 2016

Update HLT menu (V4) #15873

Merged

mmusich reviewed Sep 16, 2016

View reviewed changes

cmsbuild added tests-pending and removed tests-started labels Sep 16, 2016

cmsbuild added tests-started and removed tests-pending labels Sep 16, 2016

cmsbuild added tests-rejected and removed tests-started labels Sep 16, 2016

cmsbuild added alca-pending tests-pending and removed tests-rejected labels Sep 17, 2016

cmsbuild removed the alca-pending label Sep 17, 2016

Martin-Grunewald force-pushed the 81XHLTAfterMD2Train branch from 25266f2 to b30818f Compare September 18, 2016 07:33

cmsbuild added the alca-pending label Sep 18, 2016

cmsbuild added tests-started and removed tests-pending labels Sep 18, 2016

cmsbuild added tests-approved and removed tests-started labels Sep 18, 2016

cmsbuild added hlt-approved and removed hlt-pending labels Sep 18, 2016

cmsbuild added comparison-available and removed comparison-pending labels Sep 18, 2016

cmsbuild added alca-approved fully-signed and removed alca-pending pending-signatures labels Sep 18, 2016

cmsbuild added orp-approved and removed orp-pending labels Sep 19, 2016

cmsbuild merged commit 1348d5a into cms-sw:CMSSW_8_1_X Sep 19, 2016

mmusich mentioned this pull request Sep 20, 2016

Updating in all the relevant Global Tags the L1 menu to the version for post-TS2 data-taking: L1Menu_Collisions2016_v6r5_ugt_1board_xml #15879

Closed

Martin-Grunewald deleted the 81XHLTAfterMD2Train branch December 8, 2016 08:20

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update HLT menu (V4) [81X] #15875

Update HLT menu (V4) [81X] #15875

silviodonato commented Sep 16, 2016 •

edited

Loading

silviodonato commented Sep 16, 2016

cmsbuild commented Sep 16, 2016 •

edited

Loading

cmsbuild commented Sep 16, 2016

mmusich Sep 16, 2016

Martin-Grunewald Sep 16, 2016

mmusich Sep 16, 2016

Martin-Grunewald Sep 16, 2016

mmusich Sep 16, 2016

Martin-Grunewald Sep 16, 2016

mmusich Sep 16, 2016

mmusich Sep 16, 2016

Martin-Grunewald commented Sep 16, 2016

cmsbuild commented Sep 16, 2016 •

edited

Loading

cmsbuild commented Sep 16, 2016

Martin-Grunewald commented Sep 16, 2016

mmusich commented Sep 16, 2016

cmsbuild commented Sep 16, 2016

cmsbuild commented Sep 17, 2016

cmsbuild commented Sep 18, 2016

Martin-Grunewald commented Sep 18, 2016

cmsbuild commented Sep 18, 2016 •

edited

Loading

Martin-Grunewald commented Sep 18, 2016

cmsbuild commented Sep 18, 2016

Martin-Grunewald commented Sep 18, 2016

Martin-Grunewald commented Sep 18, 2016

cmsbuild commented Sep 18, 2016

mmusich commented Sep 18, 2016

cmsbuild commented Sep 18, 2016

davidlange6 commented Sep 19, 2016

Update HLT menu (V4) [81X] #15875

Update HLT menu (V4) [81X] #15875

Conversation

silviodonato commented Sep 16, 2016 • edited Loading

silviodonato commented Sep 16, 2016

cmsbuild commented Sep 16, 2016 • edited Loading

cmsbuild commented Sep 16, 2016

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Martin-Grunewald commented Sep 16, 2016

cmsbuild commented Sep 16, 2016 • edited Loading

cmsbuild commented Sep 16, 2016

Martin-Grunewald commented Sep 16, 2016

mmusich commented Sep 16, 2016

cmsbuild commented Sep 16, 2016

cmsbuild commented Sep 17, 2016

cmsbuild commented Sep 18, 2016

Martin-Grunewald commented Sep 18, 2016

cmsbuild commented Sep 18, 2016 • edited Loading

Martin-Grunewald commented Sep 18, 2016

cmsbuild commented Sep 18, 2016

Martin-Grunewald commented Sep 18, 2016

Martin-Grunewald commented Sep 18, 2016

cmsbuild commented Sep 18, 2016

mmusich commented Sep 18, 2016

cmsbuild commented Sep 18, 2016

davidlange6 commented Sep 19, 2016

silviodonato commented Sep 16, 2016 •

edited

Loading

cmsbuild commented Sep 16, 2016 •

edited

Loading

cmsbuild commented Sep 16, 2016 •

edited

Loading

cmsbuild commented Sep 18, 2016 •

edited

Loading