add datatier for intermediate EDM NANOAOD stuff

[arizzi]

This should solve the problem observed in production of nanoaod with intermediate edm step

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/PR-21028/1654

[cmsbuild]

A new Pull Request was created by @arizzi for master.

It involves the following packages:

Configuration/PyReleaseValidation

@cmsbuild, @GurpreetSinghChahal, @prebello, @kpedro88, @fabozzi can you please review it and eventually sign? Thanks.
@makortel, @felicepantaleo, @ebrondol, @ghellwig, @Martin-Grunewald this is something you requested to watch as well.
@davidlange6, @slava77 you are the release manager for this.

cms-bot commands are listed here

[kpedro88]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/23998/console Started: 2017/10/26 16:25

[cmsbuild]

+1
Tested at: 64b87d1
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-21028/23998/summary.html

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

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 26
• DQMHistoTests: Total histograms compared: 2815041
• DQMHistoTests: Total failures: 109
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2814761
• DQMHistoTests: Total skipped: 171
• DQMHistoTests: Total Missing objects: 0
• Checked 107 log files, 10 edm output root files, 26 DQM output files

[fabozzi]

+1

[arizzi]

@fabozzi did the workflow with this change run successfully?

…

On Fri, Oct 27, 2017 at 12:22 PM, fabozzi ***@***.***> wrote: +1 — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#21028 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEyils-TMRHyac_DxC5fHVhnXtZPDhU3ks5swa7zgaJpZM4QHch8> .

[fabozzi]

no, in pending status actually

[davidlange6]

Hi @arizzi @fabozzi from your comments- are there pending issues for this pr?

[fabozzi]

I think we should integrate with a fix to set up properly the process string of NANOMERGE step, as we learned from the injection test

[arizzi]

@fabozzi what do you mean with fix the process string? The GT cannot be "fixed" because NANOMERGE is common and any GT is fine (in fact we would not even need one)

…

On Tue, Oct 31, 2017 at 5:27 PM, fabozzi ***@***.***> wrote: I think we should integrate with a fix to set up properly the process string of NANOMERGE step, as we learned from the injection test — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#21028 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEyiloVlWHjzvlmybDXK-HQhB5FmyKAwks5sx0qBgaJpZM4QHch8> .

[fabozzi]

@arizzi
The ProcessingString is the parameter of the workflow that is used in the output dataset name. The GT of the step is copied in the ProcessingString: we need to avoid this for the NANOMERGE step and copy instead the GT of the NANOEDM step

master...fabozzi:from-CMSSW_9_4_0_pre3_nanoedmtest

I can also make a separate PR for this change in MatrixInjector

[fabozzi]

OK, I have inserted the change to MatrixInjector in the "Relval updates for 94X" (PR #21091).
This PR can go ahead as it is for me.

[arizzi]

thanks, I was trying to add it here but we had twice the same commits and from phone it was impossible

[davidlange6]

+1

[davidlange6]

merge