skims: definition, matrix, configBuilder - retry

[franzoni]

update of #9970 (closed now), forward port of #9897

++ SUMMARY
Validation and rate measuremetns on the skim producers were carried out by Alexis @alkaloge based on 7_5_0_preX

++ DETAILS

• skims DEFINITIONS of 75
• fw port of 9897; superseeds 9868 which can be closed
• rationalization of the skim matrix, intended to pass to T0 the PD => "skims to run" correspondence, in light of 2015 PD's , see https://hypernews.cern.ch/HyperNews/CMS/get/physics-announcements/3385.html
• collection of skims requirements by DPG/POG/PAGs : https://docs.google.com/spreadsheets/d/1OicL4b3jFUXalsk92GM9qLbNNAVklAGb7bf1T5BwZrY/edit#gid=0
• configBuilder modified to preserve output modules set by --eventcontent when SKIM is in the sequence

[alkaloge]

@franzoni We have a new request for skim - @emanueledimarco requested the last one here https://docs.google.com/spreadsheets/d/1OicL4b3jFUXalsk92GM9qLbNNAVklAGb7bf1T5BwZrY/edit#gid=0 to be running on Commissioning PD - Do we have time to include in this PR and backport it to 74x as well ? Of course, the matrix has to be modified as well

[cmsbuild]

A new Pull Request was created by @franzoni (Giovanni Franzoni) for CMSSW_7_5_X.

skims: definition, matrix, configBuilder - retry

It involves the following packages:

Configuration/Applications
Configuration/Skimming
DPGAnalysis/Skims

@cmsbuild, @srimanob, @boudoul, @franzoni, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @Martin-Grunewald this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[franzoni]

@alkaloge
Jack is working on that aspect. We'll intregrate when ready.
Thanks.

[franzoni]

please test

[cmsbuild]

The tests are being triggered in jenkins.

[franzoni]

+1

local tests including ALCA and all skims run fine over 5 events
The issue encountered with unscheduled https://hypernews.cern.ch/HyperNews/CMS/get/swDevelopment/3229.html (which also describes the local tests) being addressed

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs once checked with relvals in the development release cycle of CMSSW (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

[cmsbuild]

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

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs once checked with relvals in the development release cycle of CMSSW (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

[davidlange6]

+1