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

continuing HLT integrations - 75X #8753

Merged
merged 18 commits into from Apr 22, 2015

Conversation

geoff-smith
Copy link
Contributor

CMSHLT-310 - added to GRun, 50nsGRun (already in masters)
CMSHLT-273 - added to masters, 50nsGRun

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @geoff-smith for CMSSW_7_5_X.

continuing HLT integrations - 75X

It involves the following packages:

HLTrigger/Configuration

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

@cmsbuild
Copy link
Contributor

-1
Tested at: 5b1eb6e
When I ran the RelVals I found an error in the following worklfows:
135.4 step1

runTheMatrix-results/135.4_ZEE_13+ZEEFS_13+HARVESTUP15FS+MINIAODMCUP15FS/step1_ZEE_13+ZEEFS_13+HARVESTUP15FS+MINIAODMCUP15FS.log

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

@Martin-Grunewald
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

The tests are being triggered in jenkins.

@Martin-Grunewald
Copy link
Contributor

@lveldere
Backporting #8455 to 74X would be fine to keep it in sync with 74X, so we'll then also
backport this #8753 as needed.

This is even better for me as I am working on removing the HLT FAMOS cff dumps
and generate then via python from the data/fullSim cff dumps on the fly - this is in
the long run, but FastSim technical structure then being the same between 74X and
75X means I only need to solve one problem once!
This feature may require support from you and fastsim experts, but, when available, would allow
you and us to test fastsim issues without extracting the dump from the database
runnung the infamous confdb.oy (cf the issues over the last days)

@lveldere
Copy link
Contributor

Thanks Martin

On Tue, Apr 21, 2015 at 11:55 AM, Martin Grunewald <notifications@github.com

wrote:

@lveldere https://github.com/lveldere
Backporting #8455 #8455 to 74X
would be fine to keep it in sync with 74X, so we'll then also
backport this #8753 #8753 as needed.

This is even better for me as I am working on removing the HLT FAMOS cff
dumps
and generate then via python from the data/fullSim cff dumps on the fly -
this is in
the long run, but FastSim technical structure then being the same between
74X and
75X means I only need to solve one problem once!
This feature be require support from you and fastsim experts, but would
allow
you and us to test fastsim issues without extracting the dump from the
database
runnung the infamous confdb.oy (cf the issues over the last days)

That sounds great. I'd be happy to help with that when the rush for the
run2 simulations is over.


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

@cmsbuild
Copy link
Contributor

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8753/7240/summary.html

The workflows 140.53 have different files in step1_dasquery.log than the ones found in the baseline. You may want to check and retrigger the tests if necessary. You can check it in the "files" directory in the results of the comparisons

@Martin-Grunewald
Copy link
Contributor

@franzoni @civanch
Ping - could you please sign for operations?

davidlange6 added a commit that referenced this pull request Apr 22, 2015
@davidlange6 davidlange6 merged commit 507c714 into cms-sw:CMSSW_7_5_X Apr 22, 2015
@lveldere lveldere mentioned this pull request Apr 28, 2015
@Martin-Grunewald Martin-Grunewald deleted the hltIntegration75X branch June 2, 2015 12:43
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

7 participants