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

Updated GTs for 710_pre3 #2517

Merged
merged 1 commit into from
Feb 19, 2014

Conversation

rcastello
Copy link

This updates the release GT in autocond.py:

  • keeping into account the ECAL pull request Make Ecal TimeBias correction to use a new GT. #2397 , updating MC and data GT with the new TimeBias corrections
  • updating SiPixelQuality (new list of bad modules) and Templates tags for MC (postLs1) and DATA GT for run2

@cmsbuild
Copy link
Contributor

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

Updated GTs for 710_pre3

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

@cmsbuild
Copy link
Contributor

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?

@Martin-Grunewald
Copy link
Contributor

This PR is no longer accessible:

git cms-merge-topic -u 2410
No release tags specified, using default CMSSW_7_0_X_2014-02-19-0200.
You are on branch CMSSW_7_0_X
Checking out FWCore/Version in tag CMSSW_7_0_X_2014-02-19-0200.
From .

  • fed8096...647d6f4 -> merge-attempt (forced update)
    fatal: Couldn't find remote ref refs/pull/2410/head
    Unexpected end of command stream

@cmsbuild
Copy link
Contributor

-1
I ran the usual tests and I found errors in the following unit tests:

---> test runtestPhysicsToolsPatAlgos had ERRORS

you can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2517/222/summary.html

@cmsbuild
Copy link
Contributor

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

@rcastello
Copy link
Author

@cmsbuild the error has nothing to do with the code updates, it's due to the fact that this morning CMS instance of EOS was unusable. So please proceed (or re-run the test when EOS is back).
Thanks.

@nclopezo
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_1_X IBs unless changes (tests are also fine). @nclopezo, @ktf can you please take care of it?

ktf added a commit that referenced this pull request Feb 19, 2014
@ktf ktf merged commit 7f9f48f into cms-sw:CMSSW_7_1_X Feb 19, 2014
@nclopezo nclopezo modified the milestones: CMSSW_7_1_0_pre4, CMSSW_7_1_0_pre3 Feb 24, 2014
@nclopezo nclopezo modified the milestones: CMSSW_7_1_0_pre5, CMSSW_7_1_0_pre4 Mar 10, 2014
ggovi pushed a commit to ggovi/cmssw that referenced this pull request Jan 11, 2017
Update data for L1Trigger/L1TMuon
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