Clean hcal dqm

[deguio]

remove the old HCAL DQM

[deguio]

@vkhristenko

[cmsbuild]

A new Pull Request was created by @deguio (Federico De Guio) for CMSSW_8_1_X.

It involves the following packages:

DQM/HcalMonitorClient
DQM/HcalMonitorModule
DQM/HcalMonitorTasks
DQMOffline/Configuration

@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 list here #13028

[vanbesien]

please test

[cmsbuild]

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

[cmsbuild]

+1
Tested at: 34af9fd
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15374/14375/summary.html

[cmsbuild]

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

[kpedro88]

@deguio once this PR is merged, you can make a followup PR to https://github.com/cms-sw/cms-bot to remove these packages from categories.py (just to avoid clutter)

[deguio]

@kpedro88
yes I can.

[kpedro88]

@dmitrijus, @vanbesien can this also get signed before the ORP?

[dmitrijus]

+1

does this merge cleanly with #15380 ?
Either way, I will push #15380 at the orp first, before this one.

[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

[kpedro88]

@dmitrijus I think this is orthogonal with #15380, but @deguio or @vkhristenko can confirm.

[kpedro88]

@dmitrijus please at least mention this one in the ORP, even if you want to point out that the other one should be merged first in case of conflicts

[dmitrijus]

hold

@davidlange6 told me to hold it until the big HCAL is merged.

[cmsbuild]

Pull request has been put on hold by @dmitrijus
They need to issue an unhold command to remove the hold state or L1 can unhold it for all

[deguio]

@dmitrijus
fine with me. anyway the two PRs should be orthogonal.
F.

[kpedro88]

@dmitrijus #15380 is merged now, can you remove the hold?

[dmitrijus]

unhold

[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

[deguio]

@kpedro88
here we go:
cms-sw/cms-bot#733

F.