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

HLT update with new menu AND L1 update - 72X #5530

Merged
merged 18 commits into from Sep 28, 2014

Conversation

Martin-Grunewald
Copy link
Contributor

HLT update with new menu but before L1 update - 72X

This PR adds a new frozen menu for run-1 relval workflows and IB tests.
It also uses new-style naming conventions for symbolic GTs, keeping
the old one for backward compatibility.

The 73X version of this PR is #5531.

I have just updated this PR with the HLT menus to use the new L1.
Thus the update also adds the L1 PRs #5529 and #5575.
It also updates again the addOn IB tests.

Following PR #5581, we now also set useLegacyError=TRUE in StripCPE, as is done by RECO.

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @Martin-Grunewald (Martin Grunewald) for CMSSW_7_2_X.

HLT update with new menu but before L1 update

It involves the following packages:

Configuration/AlCa
Configuration/HLT
Configuration/PyReleaseValidation
HLTrigger/Configuration
Utilities/ReleaseScripts

@perrotta, @cmsbuild, @diguida, @vlimant, @cerminar, @Martin-Grunewald, @franzoni, @Dr15Jones, @rcastello, @fwyzard, @davidlange6, @ktf, @nclopezo can you please review it and eventually sign? Thanks.
@ghellwig, @wddgit 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.

@Martin-Grunewald Martin-Grunewald changed the title HLT update with new menu but before L1 update HLT update with new menu but before L1 update - 72X Sep 24, 2014
@cmsbuild
Copy link
Contributor

Pull request #5530 was updated. @perrotta, @cmsbuild, @diguida, @vlimant, @cerminar, @Martin-Grunewald, @franzoni, @Dr15Jones, @rcastello, @fwyzard, @davidlange6, @ktf, @nclopezo can you please check and sign again.

@Martin-Grunewald
Copy link
Contributor Author

Short matrix passes in 72X: 7 7 6 6 4 tests passed, 0 0 0 0 0 failed

@cmsbuild
Copy link
Contributor

Pull request #5530 was updated. @perrotta, @cmsbuild, @diguida, @vlimant, @cerminar, @Martin-Grunewald, @franzoni, @Dr15Jones, @rcastello, @fwyzard, @davidlange6, @ktf, @nclopezo can you please check and sign again.

@Martin-Grunewald
Copy link
Contributor Author

+1

@diguida
Copy link
Contributor

diguida commented Sep 28, 2014

+1

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

8 participants