add timing VS lumi (for HLT monitoring) #14937

mtosi · 2016-06-21T16:52:34Z

the main goal is to have this update in 80x for monitoring the timing vs lumi @ p5

I added a DQM source which produce the inst lumi vs LS
which has to be run w/in HLT menu
its timing should be ~neglible

then, the timing VS lumi is done by the FastTimerServiceClient

the current configuration makes the lumi profile by using SCAL,
but it can be configured in order to make use of the pixel-based luminosity as well

cmsbuild · 2016-06-21T16:52:55Z

A new Pull Request was created by @mtosi (mia tosi) for CMSSW_8_1_X.

It involves the following packages:

DQM/HLTEvF
DQM/TrackingMonitor
HLTrigger/Timer

@perrotta, @cmsbuild, @dmitrijus, @Martin-Grunewald, @fwyzard, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks.
@makortel, @Martin-Grunewald, @threus, @fioriNTU this is something you requested to watch as well.
@slava77, @Degano, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

fwyzard · 2016-06-22T00:13:25Z

@mtosi , please see the comments at #14938

fwyzard · 2016-06-22T00:14:13Z

also, once you have updated the code, could you squash all the changes in only one or two commits ?

cmsbuild · 2016-06-22T08:19:55Z

Pull request #14937 was updated. @perrotta, @cmsbuild, @dmitrijus, @Martin-Grunewald, @fwyzard, @vanbesien, @davidlange6 can you please check and sign again.

cmsbuild · 2016-06-23T09:27:15Z

Pull request #14937 was updated. @perrotta, @cmsbuild, @dmitrijus, @Martin-Grunewald, @fwyzard, @vanbesien, @davidlange6 can you please check and sign again.

Martin-Grunewald · 2016-06-23T09:43:40Z

please test

cmsbuild · 2016-06-23T09:43:58Z

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

cmsbuild · 2016-06-23T10:10:31Z

Pull request #14937 was updated. @perrotta, @cmsbuild, @dmitrijus, @Martin-Grunewald, @fwyzard, @vanbesien, @davidlange6 can you please check and sign again.

cmsbuild · 2016-06-23T12:04:34Z

-1

Tested at: cb4e0ef

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

I found follow errors while testing this PR

Failed tests: RelVals

RelVals:

When I ran the RelVals I found an error in the following worklfows:
1003.0 step3

runTheMatrix-results/1003.0_RunMinBias2012A+RunMinBias2012A+RECODDQM+HARVESTDDQM/step3_RunMinBias2012A+RunMinBias2012A+RECODDQM+HARVESTDDQM.log

10021.0 step4

runTheMatrix-results/10021.0_TenMuE_0_200+TenMuE_0_200_pythia8_2017_GenSimFull+DigiFull_2017+RecoFull_2017+HARVESTFull_2017/step4_TenMuE_0_200+TenMuE_0_200_pythia8_2017_GenSimFull+DigiFull_2017+RecoFull_2017+HARVESTFull_2017.log

1306.0 step4

runTheMatrix-results/1306.0_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15/step4_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15.log

1000.0 step4

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

1330.0 step4

runTheMatrix-results/1330.0_ZMM_13+ZMM_13+DIGIUP15+RECOUP15+HARVESTUP15/step4_ZMM_13+ZMM_13+DIGIUP15+RECOUP15+HARVESTUP15.log

9.0 step4

runTheMatrix-results/9.0_Higgs200ChargedTaus+Higgs200ChargedTaus+DIGI+RECO+HARVEST/step4_Higgs200ChargedTaus+Higgs200ChargedTaus+DIGI+RECO+HARVEST.log

25.0 step4

runTheMatrix-results/25.0_TTbar+TTbar+DIGI+RECOAlCaCalo+HARVEST+ALCATT/step4_TTbar+TTbar+DIGI+RECOAlCaCalo+HARVEST+ALCATT.log

10024.0 step4

runTheMatrix-results/10024.0_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2017_GenSimFull+DigiFull_2017+RecoFull_2017+HARVESTFull_2017/step4_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2017_GenSimFull+DigiFull_2017+RecoFull_2017+HARVESTFull_2017.log

136.731 step4

runTheMatrix-results/136.731_RunSinglePh2016B+RunSinglePh2016B+HLTDR2_2016+RECODR2_2016reHLT+HARVESTDR2/step4_RunSinglePh2016B+RunSinglePh2016B+HLTDR2_2016+RECODR2_2016reHLT+HARVESTDR2.log

50202.0 step4

runTheMatrix-results/50202.0_TTbar_13+TTbar_13+DIGIUP15_PU50+RECOUP15_PU50+HARVESTUP15_PU50/step4_TTbar_13+TTbar_13+DIGIUP15_PU50+RECOUP15_PU50+HARVESTUP15_PU50.log

25202.0 step4

runTheMatrix-results/25202.0_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15_PU25/step4_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15_PU25.log

cmsbuild · 2016-06-23T15:48:46Z

Pull request #14937 was updated. @perrotta, @cmsbuild, @dmitrijus, @Martin-Grunewald, @fwyzard, @vanbesien, @davidlange6 can you please check and sign again.

cmsbuild · 2016-06-23T15:58:50Z

Pull request #14937 was updated. @perrotta, @cmsbuild, @dmitrijus, @Martin-Grunewald, @fwyzard, @vanbesien, @davidlange6 can you please check and sign again.

cmsbuild · 2016-06-24T09:30:42Z

Pull request #14937 was updated. @perrotta, @cmsbuild, @dmitrijus, @Martin-Grunewald, @fwyzard, @vanbesien, @davidlange6 can you please check and sign again.

fwyzard · 2016-06-24T09:33:53Z

please test

cmsbuild · 2016-06-24T09:34:16Z

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

cmsbuild · 2016-06-24T12:04:39Z

+1
Tested at: b5d43f9
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-14937/13663/summary.html

cmsbuild · 2016-07-12T15:28:34Z

-1

Tested at: 0b32cce

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

I found follow errors while testing this PR

Failed tests: RelVals AddOn

RelVals:

When I ran the RelVals I found an error in the following worklfows:
4.22 step1

DAS Error

1000.0 step1

DAS Error

1001.0 step1

DAS Error

AddOn:

I found errors in the following addon tests:

cmsRun /cvmfs/cms-ib.cern.ch/week1/slc6_amd64_gcc530/cms/cmssw-patch/CMSSW_8_1_X_2016-07-12-1100/src/PhysicsTools/PatAlgos/test/IntegrationTest_cfg.py : FAILED - time: date Tue Jul 12 16:42:46 2016-date Tue Jul 12 16:41:00 2016 s - exit: 23552

mtosi · 2016-07-12T15:55:30Z

I think it is not related to this or

On 12 Jul 2016, at 17:28, cmsbuild notifications@github.com wrote:

-1

Tested at: 0b32cce

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

I found follow errors while testing this PR

Failed tests: RelVals AddOn

RelVals:
When I ran the RelVals I found an error in the following worklfows:
4.22 step1

DAS Error
1000.0 step1
DAS Error
1001.0 step1
DAS Error
AddOn:
I found errors in the following addon tests:

cmsRun /cvmfs/cms-ib.cern.ch/week1/slc6_amd64_gcc530/cms/cmssw-patch/CMSSW_8_1_X_2016-07-12-1100/src/PhysicsTools/PatAlgos/test/IntegrationTest_cfg.py : FAILED - time: date Tue Jul 12 16:42:46 2016-date Tue Jul 12 16:41:00 2016 s - exit: 23552

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

dmitrijus · 2016-07-13T13:09:50Z

please test

cmsbuild · 2016-07-13T13:10:41Z

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

cmsbuild · 2016-07-13T16:47:57Z

+1
Tested at: 0b32cce
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-14937/14027/summary.html

cmsbuild · 2016-07-14T07:48:29Z

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

perrotta · 2016-07-14T08:21:21Z

+1

cmsbuild · 2016-07-14T08:21:36Z

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

davidlange6 · 2016-07-18T10:40:44Z

+1

dmitrijus · 2016-07-19T11:54:32Z

80x: #15169

cmsbuild added this to the Next CMSSW_8_1_X milestone Jun 21, 2016

cmsbuild added dqm-pending hlt-pending pending-signatures tests-pending orp-pending comparison-pending labels Jun 21, 2016

mtosi mentioned this pull request Jun 21, 2016

add timing VS lumi (for HLT monitoring) #14938

Closed

cmsbuild added tests-started tests-pending and removed tests-pending tests-started labels Jun 23, 2016

cmsbuild added tests-started and removed tests-pending labels Jun 24, 2016

cmsbuild removed the tests-started label Jun 24, 2016

cmsbuild added tests-started and removed tests-pending labels Jul 12, 2016

cmsbuild added tests-rejected and removed tests-started labels Jul 12, 2016

cmsbuild added tests-pending and removed tests-rejected labels Jul 13, 2016

cmsbuild added tests-started and removed tests-pending labels Jul 13, 2016

cmsbuild added tests-approved and removed tests-started labels Jul 13, 2016

cmsbuild added comparison-available and removed comparison-pending labels Jul 14, 2016

cmsbuild added hlt-approved fully-signed and removed hlt-pending pending-signatures labels Jul 14, 2016

cmsbuild added orp-approved and removed orp-pending labels Jul 18, 2016

cmsbuild merged commit 53bb5b0 into cms-sw:CMSSW_8_1_X Jul 18, 2016

fwyzard mentioned this pull request Jul 18, 2016

add timing VS lumi (for HLT monitoring) #15169

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

add timing VS lumi (for HLT monitoring) #14937

add timing VS lumi (for HLT monitoring) #14937

mtosi commented Jun 21, 2016

cmsbuild commented Jun 21, 2016

fwyzard commented Jun 22, 2016

fwyzard commented Jun 22, 2016

cmsbuild commented Jun 22, 2016

cmsbuild commented Jun 23, 2016

Martin-Grunewald commented Jun 23, 2016

cmsbuild commented Jun 23, 2016 •

edited

cmsbuild commented Jun 23, 2016

cmsbuild commented Jun 23, 2016

cmsbuild commented Jun 23, 2016

cmsbuild commented Jun 23, 2016

cmsbuild commented Jun 24, 2016

fwyzard commented Jun 24, 2016

cmsbuild commented Jun 24, 2016 •

edited

cmsbuild commented Jun 24, 2016

cmsbuild commented Jul 12, 2016

mtosi commented Jul 12, 2016

dmitrijus commented Jul 13, 2016

cmsbuild commented Jul 13, 2016 •

edited

cmsbuild commented Jul 13, 2016

cmsbuild commented Jul 14, 2016

perrotta commented Jul 14, 2016

cmsbuild commented Jul 14, 2016

davidlange6 commented Jul 18, 2016

dmitrijus commented Jul 19, 2016 •

edited

add timing VS lumi (for HLT monitoring) #14937

add timing VS lumi (for HLT monitoring) #14937

Conversation

mtosi commented Jun 21, 2016

cmsbuild commented Jun 21, 2016

fwyzard commented Jun 22, 2016

fwyzard commented Jun 22, 2016

cmsbuild commented Jun 22, 2016

cmsbuild commented Jun 23, 2016

Martin-Grunewald commented Jun 23, 2016

cmsbuild commented Jun 23, 2016 • edited

cmsbuild commented Jun 23, 2016

cmsbuild commented Jun 23, 2016

cmsbuild commented Jun 23, 2016

cmsbuild commented Jun 23, 2016

cmsbuild commented Jun 24, 2016

fwyzard commented Jun 24, 2016

cmsbuild commented Jun 24, 2016 • edited

cmsbuild commented Jun 24, 2016

cmsbuild commented Jul 12, 2016

mtosi commented Jul 12, 2016

dmitrijus commented Jul 13, 2016

cmsbuild commented Jul 13, 2016 • edited

cmsbuild commented Jul 13, 2016

cmsbuild commented Jul 14, 2016

perrotta commented Jul 14, 2016

cmsbuild commented Jul 14, 2016

davidlange6 commented Jul 18, 2016

dmitrijus commented Jul 19, 2016 • edited

cmsbuild commented Jun 23, 2016 •

edited

cmsbuild commented Jun 24, 2016 •

edited

cmsbuild commented Jul 13, 2016 •

edited

dmitrijus commented Jul 19, 2016 •

edited