re-enable HLT DQM, Validation and Harvesting sequences

[mtosi]

this PR is needed to have the plots needed for the validation of the HLT menu in phase1 workflows
please, sign it as soon as possible

I tested w/ 10224.0 workflow, the corresponding DQM output is [1]

unfortunately, we realized this fix was not done yet only this afternoon :(
thanks !

[1]
/afs/cern.ch/work/t/tosi/public/STEAM/validation/2017_workflow/CMSSW_8_1_0_pre16/src/10224.0_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2017PU_GenSimFull+DigiFullPU_2017PU+RecoFullPU_2017PU+HARVESTFullPU_2017PU/DQM_V0001_R000000001__Global__CMSSW_X_Y_Z__RECO.root

[mtosi]

@Martin-Grunewald @silviodonato @dmitrijus @vanbesien @fwyzard @gennai @hengne @fabozzi

[hengne]

please test

[makortel]

Please remove the lines instead of commenting them out.

Also note that there are still additional places where DQM/VALIDATION modules are turned off because HLT not being part of 2017 workflows in the past (all can be found with git grep phase1Pixel.*FIXME).

[cmsbuild]

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

[cmsbuild]

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

It involves the following packages:

DQMOffline/Configuration
HLTriggerOffline/Common

@cmsbuild, @dmitrijus, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks.
@threus, @rociovilar this is something you requested to watch as well.
@slava77, @smuzaffar you are the release manager for this.

cms-bot commands are listed here #13028

[cmsbuild]

+1
Tested at: 836ce56
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-16576/16334/summary.html

[cmsbuild]

Comparison job queued.

[hengne]

@mtosi have you done all the cleanups?

[hengne]

please test

[cmsbuild]

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

[cmsbuild]

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

Alternative comparison was/were failed for workflow(s):
25202.0

[cmsbuild]

+1
Tested at: 6b427f8
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-16576/16339/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

[hengne]

+1

[hengne]

not sure if my signature works here... according to the labels, it is responsible by dqm group

[hengne]

otherwise computing/offline can sign it anyway, i guess.

[dmitrijus]

+1

[cmsbuild]

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, @smuzaffar

[davidlange6]

+1

[mtosi]

thanks !
mia

On Tue, Nov 15, 2016 at 3:18 PM, cmsbuild notifications@github.com wrote:

Merged #16576 #16576.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#16576 (comment), or mute the
thread
https://github.com/notifications/unsubscribe-auth/AEt581zNnaKFlOiAlfDc6lWzSLbPgi-Dks5q-b8fgaJpZM4Kxcrt
.