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

PR for HLT online DQM monitoring code for HI run in 80X #16345

Closed

Conversation

varuns23
Copy link
Contributor

The PR is made using CMSSW_8_0_20.

@mtosi @vanbesien @diguida @dmitrijus @silviodonato

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @varuns23 (varun sharma) for CMSSW_8_0_20_patchX.

It involves the following packages:

DQM/HLTEvF
DQM/Integration

@perrotta, @cmsbuild, @silviodonato, @dmitrijus, @Martin-Grunewald, @fwyzard, @vanbesien can you please review it and eventually sign? Thanks.
@threus, @batinkov this is something you requested to watch as well.
@slava77, @smuzaffar you are the release manager for this.
Note that this branch is designed for requested bug fixes specific to the CMSSW_8_0_20 release.
If you wish to make a pull request for the CMSSW_8_0_X release cycle, please use the CMSSW_8_0_X branch instead

cms-bot commands are listed here #13028

@silviodonato
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

The tests are being triggered in jenkins.

@diguida
Copy link
Contributor

diguida commented Oct 25, 2016

@varuns23 two comments:

  • Please make a PR against the vanilla CMSSW_8_0_X branch
  • In order this to be scrutinised and accepted, please make a corresponding PR in the development branch, namely CMSSW_8_1_X

@diguida
Copy link
Contributor

diguida commented Oct 25, 2016

Comment to self: the PR in 8.1.X is #16346
So, please make a PR for vanilla CMSSW_8_0_X using the very same branch varuns23:hlt-onlineDQM-HI-80X

@fwyzard
Copy link
Contributor

fwyzard commented Oct 26, 2016 via email

@varuns23
Copy link
Contributor Author

Already made new one: #16355

Closing this PR.

@varuns23 varuns23 closed this Oct 26, 2016
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.

None yet

5 participants