Fix for crash in 140.0: remove PostLS1_HI customization from legacy HI workflow

[kkrajczar]

Dear all,

The purpose of this PR is to fix a crash [1] arising from the mixing of the Run-2 conditions and the Run-1 GT. In this PR, we remove the postLS1_HI customization and switch the HLT menu from HIon (which relied on the PostLS1_HI customization) to the fake HLT menu in workflow 140.0. In my tests, this PR solves the crash.

Cheers,
Krisztian

[1] An exception of category 'CorrelatedNoisifier' occurred while
[0] Processing run: 1 lumi: 36 event: 36
[1] Running path 'digitisation_step'
[2] Calling event method for module CSCDigiProducer/'simMuonCSCDigis'

[cmsbuild]

A new Pull Request was created by @kkrajczar (Krisztian Krajczar) for CMSSW_7_5_X.

Fix for crash in 140.0: remove PostLS1_HI customization from legacy HI workflow

It involves the following packages:

Configuration/PyReleaseValidation

@cmsbuild, @srimanob, @boudoul, @franzoni can you please review it and eventually sign? Thanks.
@ghellwig, @Martin-Grunewald 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.

[franzoni]

hello @kkrajczar ,
with the wf 140 being in full a runI workflow, your change is a correction of a mistake (thanks).

However, is there any wf for HIN in runII conditons ? Can you please make a proposal - we'd need to keep the new centre of mass energy, geomety, AlCal, and customize function validated for runII.

Cheers and thanks,
g.

[franzoni]

please test

[cmsbuild]

The tests are being triggered in jenkins.

[kkrajczar]

Hi Giovanni,

We have a run2 workflow with new center-of-mass energy, geometry, etc: wf 145; it is already in the matrix.

Cheers,
Krisztian

-------- Original message --------
From: Giovanni Franzoni
Date:12/06/2015 15:45 (GMT+01:00)
To: cms-sw/cmssw
Cc: Krisztian Krajczar
Subject: Re: [cmssw] Fix for crash in 140.0: remove PostLS1_HI customization from legacy HI workflow (#9487)

hello @kkrajczarhttps://github.com/kkrajczar ,
with the wf 140 being in full a runI workflow, your change is a correction of a mistake (thanks).

However, is there any wf for HIN in runII conditons ? Can you please make a proposal - we'd need to keep the new centre of mass energy, geomety, AlCal, and customize function validated for runII.

Cheers and thanks,
g.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/9487#issuecomment-111497029.

[cmsbuild]

+1
Tested at: 2415721
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9487/2566/summary.html

[franzoni]

+1

[cmsbuild]

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

[davidlange6]

+1