Fixing AlCa Matrix for 2015 EOY Reprocessing

[mmusich]

Fixing AlCa Matrix for the EOY 2015 reprocessing.
As in: https://twiki.cern.ch/twiki/bin/viewauth/CMS/AlCaRecoMatrix#25_ns_7e33_collisions_13_TeV

[mmusich]

please test

[cmsbuild]

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

[cmsbuild]

A new Pull Request was created by @mmusich (Marco Musich) for CMSSW_7_6_X.

It involves the following packages:

Configuration/AlCa

@diguida, @cerminar, @cmsbuild, @franzoni, @mmusich, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @Martin-Grunewald this is something you requested to watch as well.
@slava77, @Degano, @smuzaffar you are the release manager for this.

Following commands in first line of a comment are recognized

• +1|approve[d]|sign[ed]: L1/L2's to approve it
• -1|reject[ed]: L1/L2's to reject it
• assign <category>[,<category>[,...]]: L1/L2's to request signatures from other categories
• unassign <category>[,<category>[,...]]: L1/L2's to remove signatures from other categories
• hold: L1/all L2's/release manager to mark it as on hold
• unhold: L1/user who put this PR on hold
• merge: L1/release managers to merge this request
• [@cmsbuild,] please test: L1/L2 and selected users to start jenkins tests
• [@cmsbuild,] please test with cms-sw/cmsdist#<PR>: L1/L2 and selected users to start jenkins tests using externals from cmsdist

[cmsbuild]

-1
Tested at: 76cfccc
When I ran the RelVals I found an error in the following worklfows:
1000.0 step2

runTheMatrix-results/1000.0_RunMinBias2011A+RunMinBias2011A+TIER0+SKIMD+HARVESTDfst2+ALCASPLIT/step2_RunMinBias2011A+RunMinBias2011A+TIER0+SKIMD+HARVESTDfst2+ALCASPLIT.log

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

[cmsbuild]

Pull request #12716 was updated. @diguida, @cerminar, @cmsbuild, @franzoni, @mmusich, @davidlange6 can you please check and sign again.

[mmusich]

please test

[cmsbuild]

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

[cmsbuild]

-1
Tested at: a597e95
When I ran the RelVals I found an error in the following worklfows:
1000.0 step2

runTheMatrix-results/1000.0_RunMinBias2011A+RunMinBias2011A+TIER0+SKIMD+HARVESTDfst2+ALCASPLIT/step2_RunMinBias2011A+RunMinBias2011A+TIER0+SKIMD+HARVESTDfst2+ALCASPLIT.log
----- Begin Fatal Exception 09-Dec-2015 09:54:19 CET-----------------------
An exception of category 'Configuration' occurred while
   [0] Processing run: 165121 lumi: 62 event: 23609118
   [1] Running path 'pathALCARECOHcalCalIterativePhiSym'
   [2] Calling event method for module HLTHighLevel/'hcalphisymHLT'
Exception Message:
 HLTHighLevel [instance: hcalphisymHLT - path: pathALCARECOHcalCalIterativePhiSym]: No triggerList with key HcalCalIterativePhiSym in AlCaRecoTriggerBitsRcd
----- End Fatal Exception -------------------------------------------------

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

[cmsbuild]

Pull request #12716 was updated. @diguida, @cerminar, @cmsbuild, @franzoni, @mmusich, @davidlange6 can you please check and sign again.

[mmusich]

please test

[cmsbuild]

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

[cmsbuild]

+1
Tested at: 5a2ecf5
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12716/10209/summary.html

[mmusich]

+1
few AlCaReco streams for Prompt had to be removed due to missing AlCaRecoTriggerBit in the run used for RelVal tests. Complete set + new Global Tag will be proposed for 8.0.X.

[cmsbuild]

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

[cmsbuild]

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

[davidlange6]

+1

[smuzaffar]

Looks like automatic forward port of it broke 80X IBs
https://cmssdt.cern.ch/SDT/cgi-bin/buildlogs/slc6_amd64_gcc493/CMSSW_8_0_THREADED_X_2015-12-09-2000/pyRelValMatrixLogs/run/4.29_RunMinBias2011B+RunMinBias2011B+HLTD+RECODreHLT+HARVESTDreHLT+SKIMDreHLT/step3_RunMinBias2011B+RunMinBias2011B+HLTD+RECODreHLT+HARVESTDreHLT+SKIMDreHLT.log

[smuzaffar]

@davidlange6 , should we revert it for 80X?

[mmusich]

@smuzaffar yes please, I need in any case to modify it for 80X

[davidlange6]

looks like we should unless @mmusich has a quick comment? Its not really clear to me what went wrong.

On Dec 10, 2015, at 9:18 AM, Malik Shahzad Muzaffar notifications@github.com wrote:

@davidlange6 , should we revert it for 80X?

—
Reply to this email directly or view it on GitHub.

[mmusich]

@davidlange6 at the moment is not clear to me either. I need anyway to introduce more changes for 80X so please revert it.