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

Cluster multiplicity74 x #11338

Merged
merged 3 commits into from Oct 10, 2015
Merged

Cluster multiplicity74 x #11338

merged 3 commits into from Oct 10, 2015

Conversation

fioriNTU
Copy link
Contributor

retune of bundaries between cluster multiplicity regions

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @fioriNTU for CMSSW_7_4_X.

Cluster multiplicity74 x

It involves the following packages:

DQM/SiStripMonitorCluster
DQM/TrackingMonitorClient

@cmsbuild, @danduggan, @broen, @deguio can you please review it and eventually sign? Thanks.
@makortel, @threus 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.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@deguio
Copy link
Contributor

deguio commented Sep 23, 2015

please test

@deguio
Copy link
Contributor

deguio commented Sep 23, 2015

+1

@cmsbuild
Copy link
Contributor

The tests are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs after it passes the integration tests and once validation in the development release cycle CMSSW_7_6_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs after it passes the integration tests and once validation in the development release cycle CMSSW_7_6_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_7_6_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_7_6_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

@fioriNTU
Copy link
Contributor Author

Hi, I wonder why this PR is not yet merged, can somebody clarify?
Also the equivalent PR for 75x is not yet merged.

FF

@deguio
Copy link
Contributor

deguio commented Sep 28, 2015

each change needs to be merged in 76 and go through a full validation cycle there before being merged in other closed cycles.

@fioriNTU
Copy link
Contributor Author

Hi Federico,
actually this change has been merged in 76x, it is PR #11341. Moreover
this PR is only a change of two parameters in a cfi file, should not
give any problem.
Thanks anyway for fast feedback.

FF

Il 28/09/2015 16.08, Federico De Guio ha scritto:

each change needs to be merged in 76 and go through a full validation
cycle there before being merged in other closed cycles.


Reply to this email directly or view it on GitHub
#11338 (comment).


Questa e-mail è stata controllata per individuare virus con Avast antivirus.
https://www.avast.com/antivirus

@davidlange6
Copy link
Contributor

+1

cmsbuild added a commit that referenced this pull request Oct 10, 2015
@cmsbuild cmsbuild merged commit 864af8e into cms-sw:CMSSW_7_4_X Oct 10, 2015
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

4 participants