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

Remove HLT-like customisations from RelVal step for HI Run2 MC #12648

Merged

Conversation

diguida
Copy link
Contributor

@diguida diguida commented Dec 2, 2015

The run2_mc_HIon key is added to the list of available GlobalTags for testing purposes, originally in autoCond, by autoCondHLT, so it is specific for TSG tests, like addOn.
This key is now removed in RelVal steps and local tests. It is in fact kept for HLT tests.

…eps and local tests. They are in fact kept for HLT tests.
@cmsbuild
Copy link
Contributor

cmsbuild commented Dec 2, 2015

A new Pull Request was created by @diguida (Salvatore Di Guida) for CMSSW_7_5_X.

It involves the following packages:

Configuration/PyReleaseValidation
RecoHI/HiCentralityAlgos

@cvuosalo, @boudoul, @franzoni, @fabozzi, @cmsbuild, @srimanob, @slava77, @hengne, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @azsigmon, @jazzitup, @richard-cms, @Martin-Grunewald, @yenjie, @kurtejung, @istaslis, @mandrenguyen, @dgulhan, @yetkinyilmaz this is something you requested to watch as well.
@slava77, @Degano, @smuzaffar you are the release manager for this.

Following commands in first line of a comment are recognized

  • +1|approve[d]|sign[ed]: L1/L2's to approve it
  • -1|reject[ed]: L1/L2's to reject it
  • assign <category>[,<category>[,...]]: L1/L2's to request signatures from other categories
  • unassign <category>[,<category>[,...]]: L1/L2's to remove signatures from other categories
  • hold: L1/all L2's/release manager to mark it as on hold
  • unhold: L1/user who put this PR on hold
  • merge: L1/release managers to merge this request
  • [@cmsbuild,] please test: L1/L2 and selected users to start jenkins tests
  • [@cmsbuild,] please test with cms-sw/cmsdist#<PR>: L1/L2 and selected users to start jenkins tests using externals from cmsdist

@diguida
Copy link
Contributor Author

diguida commented Dec 2, 2015

@mmusich this is something you want to watch as well.

@diguida
Copy link
Contributor Author

diguida commented Dec 2, 2015

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Dec 2, 2015

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/10101/console

@diguida
Copy link
Contributor Author

diguida commented Dec 2, 2015

assign alca

@cmsbuild
Copy link
Contributor

cmsbuild commented Dec 2, 2015

-1
Tested at: 393d02a
When I ran the RelVals I found an error in the following worklfows:
1001.0 step2

runTheMatrix-results/1001.0_RunMinBias2011A+RunMinBias2011A+TIER0EXP+ALCAEXP+ALCAHARVD1+ALCAHARVD2+ALCAHARVD3+ALCAHARVD4/step2_RunMinBias2011A+RunMinBias2011A+TIER0EXP+ALCAEXP+ALCAHARVD1+ALCAHARVD2+ALCAHARVD3+ALCAHARVD4.log
----- Begin Fatal Exception 02-Dec-2015 19:16:57 CET-----------------------
An exception of category 'FallbackFileOpenError' occurred while
   [0] Constructing the EventProcessor
   [1] Constructing input source of type PoolSource
   [2] Calling RootFileSequenceBase::initTheFile()
   [3] Calling StorageFactory::open()
   [4] Calling XrdFile::open()
Exception Message:
Failed to open the file 'root://xrootd-cms.infn.it//store/data/Run2011A/MinimumBias/RAW/v1/000/165/121/0699429A-B37F-E011-A57A-0019B9F72D71.root'
   Additional Info:
      [a] XrdCl::File::Open(name='root://eoscms.cern.ch//eos/cms/store/data/Run2011A/MinimumBias/RAW/v1/000/165/121/0699429A-B37F-E011-A57A-0019B9F72D71.root', flags=0x10, permissions=0660) => error '[FATAL] Connection error' (errno=0, code=108)
      [b] Remote server already encountered a fatal error; no redirections were performed.
      [c] Input file root://eoscms.cern.ch//eos/cms/store/data/Run2011A/MinimumBias/RAW/v1/000/165/121/0699429A-B37F-E011-A57A-0019B9F72D71.root could not be opened.
Fallback Input file root://xrootd-cms.infn.it//store/data/Run2011A/MinimumBias/RAW/v1/000/165/121/0699429A-B37F-E011-A57A-0019B9F72D71.root also could not be opened.
Original exception info is above; fallback exception info is below.
      [d] XrdCl::File::Open(name='root://xrootd-cms.infn.it//store/data/Run2011A/MinimumBias/RAW/v1/000/165/121/0699429A-B37F-E011-A57A-0019B9F72D71.root', flags=0x10, permissions=0660) => error '[ERROR] Server responded with an error: [3010] Unable to open /store/data/Run2011A/MinimumBias/RAW/v1/000/165/121/0699429A-B37F-E011-A57A-0019B9F72D71.root; Permission denied
' (errno=3010, code=400). No additional data servers were found.
      [e] Last URL tried: root://node12.datagrid.cea.fr:11001//store/data/Run2011A/MinimumBias/RAW/v1/000/165/121/0699429A-B37F-E011-A57A-0019B9F72D71.root?tried=
      [f] Problematic data server: node12.datagrid.cea.fr:11001
      [g] Disabled source: node12.datagrid.cea.fr:11001
----- End Fatal Exception -------------------------------------------------

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

@diguida
Copy link
Contributor Author

diguida commented Dec 2, 2015

test failure unrelated to this PR

@diguida
Copy link
Contributor Author

diguida commented Dec 2, 2015

please test
Let's try again

@cmsbuild
Copy link
Contributor

cmsbuild commented Dec 2, 2015

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/10105/console

@cmsbuild
Copy link
Contributor

cmsbuild commented Dec 2, 2015

@cmsbuild
Copy link
Contributor

cmsbuild commented Dec 2, 2015

@slava77
Copy link
Contributor

slava77 commented Dec 3, 2015

+1

for #12648 393d02a

  • code changes are as described; reco is minimally affected: only in (unused?) test files in RecoHI/HiCentralityAlgos
  • jenkins tests pass and comparisons with baseline show no differences as expected

@fabozzi
Copy link
Contributor

fabozzi commented Dec 3, 2015

+1

@cmsbuild
Copy link
Contributor

cmsbuild commented Dec 3, 2015

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) and once validation in the development release cycle CMSSW_7_6_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar

@davidlange6
Copy link
Contributor

+1

cmsbuild added a commit that referenced this pull request Dec 8, 2015
…yIonMC

Remove HLT-like customisations from RelVal step for HI Run2 MC
@cmsbuild cmsbuild merged commit 2b59e13 into cms-sw:CMSSW_7_5_X Dec 8, 2015
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