Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ECAL DQM harvesting migration and online bugfix #5786

Merged
merged 1 commit into from
Oct 15, 2014
Merged

ECAL DQM harvesting migration and online bugfix #5786

merged 1 commit into from
Oct 15, 2014

Conversation

yiiyama
Copy link
Contributor

@yiiyama yiiyama commented Oct 11, 2014

The branch name is not quite correct for what this PR is. I'm sorry. This PR synchronizes ECAL DQM with the HEAD of 73X.

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @yiiyama (Yutaro Iiyama) for CMSSW_7_2_X.

Synching 72 with 73

It involves the following packages:

DQM/EcalCommon
DQM/EcalMonitorClient
DQM/EcalMonitorDbModule
DQM/EcalMonitorTasks
DQMOffline/Configuration
DQMOffline/Ecal

The following packages do not have a category, yet:

DQM/EcalMonitorClient
DQM/EcalMonitorDbModule
DQM/EcalMonitorTasks

@nclopezo, @danduggan, @rovere, @cmsbuild, @deguio, @ojeda can you please review it and eventually sign? Thanks.
@argiro, @threus, @rociovilar this is something you requested to watch as well.
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.

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@deguio
Copy link
Contributor

deguio commented Oct 13, 2014

+1
this is the backport of a fix developed on top of 73. the migration to the DQMEDHarvester is also backported.

@davidlange6
Copy link
Contributor

please change the name of this PR to a meaningful one reflecting it contents for the release notes (and the similar one) if you want it considered for integration. Thanks

@yiiyama yiiyama changed the title Synching 72 with 73 ECAL DQM harvesting migration and online bugfix Oct 14, 2014
@yiiyama
Copy link
Contributor Author

yiiyama commented Oct 14, 2014

@davidlange6 The title reflects the content now. Thank you.

davidlange6 added a commit that referenced this pull request Oct 15, 2014
ECAL DQM harvesting migration and online bugfix
@davidlange6 davidlange6 merged commit 56ca13c into cms-sw:CMSSW_7_2_X Oct 15, 2014
@yiiyama yiiyama deleted the ecaldqm-harvesting-bugfix branch October 15, 2014 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants