JsonMonitoring fixes for 7_2_X

[smorovic]

Changes:

• output_X.jsd file name and path as hltd expects
• making sure to write ini file at first seen lumisection in each process

[cmsbuild]

A new Pull Request was created by @smorovic (Srecko Morovic) for CMSSW_7_2_X.

JsonMonitoring fixes for 7_2_X

It involves the following packages:

EventFilter/Utilities
HLTrigger/JSONMonitoring

@perrotta, @cmsbuild, @nclopezo, @emeschi, @Martin-Grunewald, @fwyzard, @mommsen can you please review it and eventually sign? Thanks.
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.
@nclopezo you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[Martin-Grunewald]

Remember to make a PR for 73X as well - in fact, there is #6105 already (previous round) - still not signed by DAQ.

[smorovic]

Good point. We first need to get #6105 merged - or Aram could also add this commit to it.

[avetisya]

I've added this commit to #6105 .

[emeschi]

+1
On 04 Nov 2014, at 17:51, cmsbuild <notifications@github.commailto:notifications@github.com> wrote:

A new Pull Request was created by @smorovichttps://github.com/smorovic (Srecko Morovic) for CMSSW_7_2_X.

JsonMonitoring fixes for 7_2_X

It involves the following packages:

EventFilter/Utilities
HLTrigger/JSONMonitoring

@perrottahttps://github.com/perrotta, @cmsbuildhttps://github.com/cmsbuild, @nclopezohttps://github.com/nclopezo, @emeschihttps://github.com/emeschi, @Martin-Grunewaldhttps://github.com/Martin-Grunewald, @fwyzardhttps://github.com/fwyzard, @mommsenhttps://github.com/mommsen can you please review it and eventually sign? Thanks.
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.
@nclopezohttps://github.com/nclopezo you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/6203#issuecomment-61672392.

[cmsbuild]

+1
Tested at: 523f109
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6203/633/summary.html

[cmsbuild]

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

[Martin-Grunewald]

+1

[Martin-Grunewald]

Please also put this latest change into 73!

[cmsbuild]

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

[smorovic]

I made the 73X PR: #6243

[davidlange6]

+1