add PV monitoring @HLT

[cmsbuild]

we were missing some vertex collection in the validation
and the monitoring for all collection (both online and offline)

this PR adds the validation and monitoring for vertex @HLT

there is a missing collection in the HLT output modules (hltVerticesPFFilter)
I opened the JIRA ticket #1201 for adding it

I'm currently running runTheMatrix -l limited
Automatically ported from CMSSW_9_0_X #17667 (original by @mtosi).
Please wait for a new IB (12 to 24H) before requesting to test this PR.

[cmsbuild]

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

It involves the following packages:

DQM/HLTEvF
DQMOffline/Trigger
Validation/RecoVertex

@perrotta, @cmsbuild, @silviodonato, @dmitrijus, @Martin-Grunewald, @fwyzard, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks.
@battibass, @makortel, @felicepantaleo, @GiacomoSguazzoni, @jhgoh, @VinInn, @calderona, @HuguesBrun, @rovere, @mtosi, @rociovilar, @trocino, @dgulhan, @venturia this is something you requested to watch as well.
@Muzaffar, @davidlange6, @smuzaffar you are the release manager for this.

cms-bot commands are listed here #13028

[Martin-Grunewald]

@mtosi
Please fix or close....

[cmsbuild]

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

[mtosi]

sorry the delay, but the issue w/ frontier and the weekend prevented to fix it sooner

there was a bug in the original PV monitoring code,
which is fixed, now

hope this PR is fine, now

[dmitrijus]

+1

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/18205/console Started: 2017/03/07 13:52

[cmsbuild]

-1

Tested at: 085be0e

You can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-17693/18205/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:
1330.0 step3

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

10021.0 step3

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

136.731 step3

runTheMatrix-results/136.731_RunSinglePh2016B+RunSinglePh2016B+HLTDR2_2016+RECODR2_2016reHLT_skimSinglePh_HIPM+HARVESTDR2/step3_RunSinglePh2016B+RunSinglePh2016B+HLTDR2_2016+RECODR2_2016reHLT_skimSinglePh_HIPM+HARVESTDR2.log

10024.0 step3

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

10824.0 step3

runTheMatrix-results/10824.0_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2018_GenSimFull+DigiFull_2018+RecoFull_2018+ALCAFull_2018+HARVESTFull_2018/step3_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2018_GenSimFull+DigiFull_2018+RecoFull_2018+ALCAFull_2018+HARVESTFull_2018.log

25202.0 step3

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

[cmsbuild]

Comparison not run due to runTheMatrix errors (RelVals and Igprof tests were also skipped)

[cmsbuild]

Pull request #17693 was updated. @perrotta, @cmsbuild, @civanch, @monttj, @ghellwig, @dmitrijus, @silviodonato, @arunhep, @mdhildreth, @fabozzi, @Martin-Grunewald, @franzoni, @kpedro88, @cerminar, @slava77, @GurpreetSinghChahal, @fwyzard, @mmusich, @vanbesien, @davidlange6 can you please check and sign again.

[cmsbuild]

Pull request #17693 was updated. @perrotta, @cmsbuild, @civanch, @monttj, @ghellwig, @dmitrijus, @silviodonato, @arunhep, @mdhildreth, @fabozzi, @Martin-Grunewald, @franzoni, @kpedro88, @cerminar, @slava77, @GurpreetSinghChahal, @fwyzard, @mmusich, @vanbesien, @davidlange6 can you please check and sign again.

[Martin-Grunewald]

-1
merge conflicts

[cmsbuild]

Pull request #17693 was updated. @perrotta, @cmsbuild, @civanch, @monttj, @ghellwig, @dmitrijus, @silviodonato, @arunhep, @mdhildreth, @fabozzi, @Martin-Grunewald, @franzoni, @kpedro88, @cerminar, @slava77, @GurpreetSinghChahal, @fwyzard, @mmusich, @vanbesien, @davidlange6 can you please check and sign again.

[Martin-Grunewald]

-1
still merge conflicts

[cmsbuild]

Pull request #17693 was updated. @perrotta, @cmsbuild, @civanch, @monttj, @ghellwig, @dmitrijus, @silviodonato, @arunhep, @mdhildreth, @fabozzi, @Martin-Grunewald, @franzoni, @kpedro88, @cerminar, @slava77, @GurpreetSinghChahal, @fwyzard, @mmusich, @vanbesien, @davidlange6 can you please check and sign again.

[silviodonato]

Ciao Mia,
you still need to rebase this 91X PR (please have a look to https://github.com/cms-sw/cmssw/pull/17693/files).

[cmsbuild]

Pull request #17693 was updated. @perrotta, @cmsbuild, @civanch, @monttj, @ghellwig, @dmitrijus, @silviodonato, @arunhep, @mdhildreth, @fabozzi, @Martin-Grunewald, @franzoni, @kpedro88, @cerminar, @slava77, @GurpreetSinghChahal, @fwyzard, @mmusich, @vanbesien, @davidlange6 can you please check and sign again.

[mtosi]

PR #17996 opened
I would suggest to close this one
(I cannot do it, because I did not open it)

[silviodonato]

Could you close this PR? (#17996)