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

GTs for pre9: migrating accounts to prod DB and updating muon layer alignment #4123

Merged
merged 1 commit into from Jun 6, 2014

Conversation

rcastello
Copy link

  • Migrating few account in production DB (and thus removing the "PRE" prefix from the GT name)
  • Updating muon geometry for data GT fro Run1 having positions of a few internal layers of muon chambers corrected.
  • queued for 710_pre9

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 5, 2014

A new Pull Request was created by @rcastello (Roberto Castello) for CMSSW_7_1_X.

migrating accounts to prod DB and updating muon layer alignment

It involves the following packages:

Configuration/AlCa

@cmsbuild, @Degano, @diguida, @rcastello, @nclopezo can you please review it and eventually sign? Thanks.
@ghellwig 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, @ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@rcastello
Copy link
Author

+1

@rcastello rcastello changed the title migrating accounts to prod DB and updating muon layer alignment GT for pre9: migrating accounts to prod DB and updating muon layer alignment Jun 5, 2014
@rcastello rcastello changed the title GT for pre9: migrating accounts to prod DB and updating muon layer alignment GTs for pre9: migrating accounts to prod DB and updating muon layer alignment Jun 5, 2014
@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 5, 2014

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_1_X IBs unless changes or unless it breaks tests. @nclopezo, @ktf can you please take care of it?

@diguida
Copy link
Contributor

diguida commented Jun 5, 2014

@apfeiffer1 As agreed, please migrate to the new Conditions the GTs in this PR and update the BOOSTIO PRs: #4003 for 71X and #4013 for 72X (once forward porting is done).
Thanks

@apfeiffer1
Copy link
Contributor

Hi Salvatore, all,

@apfeiffer1 https://github.com/apfeiffer1 As agreed, please migrate to

the new Conditions the GTs in this PR and update the BOOSTIO PRs: #4003
#4003 for 71X and #4013
#4013 for 72X (once forward porting
is done).

yes, will do. ETA an hour or two as I've to run an errand right now ...

Thanks,
cheers, andreas

@davidlange6
Copy link
Contributor

@ktf, @nclopezo @Degano - can you launch the comparisons
@slava77 - fyi

@apfeiffer1
Copy link
Contributor

The new PR for the new Conditions DB switch to go with 710pre9 is #4125.

Thanks,
cheers, andreas

@diguida
Copy link
Contributor

diguida commented Jun 6, 2014

@ktf @nclopezo @Degano are the tests ready?
Thanks

@ktf
Copy link
Contributor

ktf commented Jun 6, 2014

I'm running them by hand right now… Build infrastructure is a bit overloaded.

@diguida
Copy link
Contributor

diguida commented Jun 6, 2014

@ktf Thanks for your support!

ktf added a commit that referenced this pull request Jun 6, 2014
GTs for pre9: migrating accounts to prod DB and updating muon layer alignment
@ktf ktf merged commit 68c5833 into cms-sw:CMSSW_7_1_X Jun 6, 2014
@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 6, 2014

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 6, 2014

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

6 participants