Fix for heavyIon object in both embedded and non-emb. HI samples

[yetkinyilmaz]

Running pdigi_hi for Hydjet samples so that the input for the heavyIon object is produced which is (CrossingFrame) by the same module in both embedded and non-embedded HI workflows.

pdigi_hi is added, and the relval workflows adjusted accordingly.

Tested workflows: 140, 145, 300, 140.53, in addition to those in runTheMatrix.py -s

Needed for RelVals, and especially needed for the release for MC DIGI campaign.

[cmsbuild]

A new Pull Request was created by @yetkinyilmaz for CMSSW_7_5_X.

It involves the following packages:

Configuration/PyReleaseValidation
Configuration/StandardSequences
SimGeneral/MixingModule

@civanch, @boudoul, @franzoni, @mdhildreth, @fabozzi, @cmsbuild, @srimanob, @hengne, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @wmtan, @makortel, @GiacomoSguazzoni, @rovere, @VinInn, @Martin-Grunewald, @cerati, @dgulhan 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

[diguida]

please test

[cmsbuild]

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

[cmsbuild]

-1
Tested at: bbe27f9
When I ran the RelVals I found an error in the following worklfows:
140.53 step1

DAS Error

1000.0 step1

DAS Error

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

[hengne]

please test

DAS error can be temporary.

[hengne]

please test

[cmsbuild]

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

[cmsbuild]

Pull request #12660 was updated. @civanch, @boudoul, @franzoni, @mdhildreth, @fabozzi, @cmsbuild, @srimanob, @deguio, @vanbesien, @hengne, @davidlange6 can you please check and sign again.

[yetkinyilmaz]

I have merged @rkunnawa's fixes for the segfault in jet validation.

[diguida]

please test

[cmsbuild]

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

[cmsbuild]

+1
Tested at: c80b000
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12660/10149/summary.html

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

[cmsbuild]

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

[hengne]

+1

[civanch]

+1

[deguio]

changes are fine for DQM, but are there corresponding PR for 76 and 80?

[yetkinyilmaz]

@deguio , this PR is only relevant for CMSSW_7_5_X, because the mixing workflows in the other releases are different. Those releases do need updates as well, however it is not this bugfix, there will have to be more work for those.

[yetkinyilmaz]

Hi. Can the remaining signatures comment? We need this in CMSSW_7_5_7_patch4.

[deguio]

+1