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

ProductNotFound exception in AlcaPCCIntegrator in workflow 1001.2 #37685

Closed
makortel opened this issue Apr 26, 2022 · 14 comments · Fixed by #37712
Closed

ProductNotFound exception in AlcaPCCIntegrator in workflow 1001.2 #37685

makortel opened this issue Apr 26, 2022 · 14 comments · Fixed by #37712

Comments

@makortel
Copy link
Contributor

Workflow 1001.2 step 2 is failing in CMSSW_12_4_X_2022-04-26-1100 with

----- Begin Fatal Exception 26-Apr-2022 13:15:01 CEST-----------------------
An exception of category 'ProductNotFound' occurred while
   [0] Processing  Event run: 305064 lumi: 2 event: 1783800 stream: 0
   [1] Running path 'pathALCARECOAlCaPCCRandom'
   [2] Calling method for module AlcaPCCIntegrator/'alcaPCCIntegratorRandom'
Exception Message:
Principal::getByToken: Found zero products matching all criteria
Looking for type: reco::PixelClusterCountsInEvent
Looking for module label: hltAlcaPixelClusterCounts
Looking for productInstanceName: alcaPCCEvent

   Additional Info:
      [a] If you wish to continue processing events after a ProductNotFound exception,
add "SkipEvent = cms.untracked.vstring('ProductNotFound')" to the "options" PSet in the configuration.

----- End Fatal Exception -------------------------------------------------

https://cmssdt.cern.ch/SDT/cgi-bin/logreader/slc7_amd64_gcc10/CMSSW_12_4_X_2022-04-26-1100/pyRelValMatrixLogs/run/1001.2_RunZeroBias2017F+RunZeroBias2017F+TIER0EXPRUN2+ALCAEXPRUN2+ALCAHARVDSIPIXELCAL+ALCAHARVDSIPIXELCALLA/step2_RunZeroBias2017F+RunZeroBias2017F+TIER0EXPRUN2+ALCAEXPRUN2+ALCAHARVDSIPIXELCAL+ALCAHARVDSIPIXELCALLA.log#/

@makortel
Copy link
Contributor Author

assign alca

@cmsbuild
Copy link
Contributor

New categories assigned: alca

@yuanchao,@francescobrivio,@malbouis,@tvami you have been requested to review this Pull request/Issue and eventually sign? Thanks

@cmsbuild
Copy link
Contributor

A new Issue was created by @makortel Matti Kortelainen.

@Dr15Jones, @perrotta, @dpiparo, @makortel, @smuzaffar, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@makortel
Copy link
Contributor Author

#37590 looks like a potential candidate for the cause

@tvami
Copy link
Contributor

tvami commented Apr 26, 2022

#37590 looks like a potential candidate for the cause

Yes, but I think the solution is not really to modify the content of #37590
but to modify the wf 1001.2

Right now it uses 2017 data
https://github.com/cms-sw/cmssw/blob/master/Configuration/PyReleaseValidation/python/relval_production.py#L15
but I think that should be moved to use the 2021 data (which then will have the needed collection this issue is reporting)

@tvami
Copy link
Contributor

tvami commented Apr 26, 2022

assign pdmv

@cmsbuild
Copy link
Contributor

New categories assigned: pdmv

@bbilin,@jordan-martins,@kskovpen you have been requested to review this Pull request/Issue and eventually sign? Thanks

@mmusich
Copy link
Contributor

mmusich commented Apr 27, 2022

Sorry what's the plan to fix this?
If the PCC workflow is not appropriate for 2017 can't it just be removed?
Or even better, why is it not customized for different inputs as a function of the era?

@tvami
Copy link
Contributor

tvami commented Apr 27, 2022

In the end I resolved it inside the AlCa jurisdiction by not adding this stream to the allForExpress

@mmusich
Copy link
Contributor

mmusich commented Apr 27, 2022

@tvami I think #37712 might be also a viable solution, allowing you to remain with one list for all eras.

@tvami
Copy link
Contributor

tvami commented Apr 27, 2022

Yes, that's nice, thanks Marco!

@tvami
Copy link
Contributor

tvami commented Apr 27, 2022

unassign pdmv

@tvami
Copy link
Contributor

tvami commented Apr 28, 2022

+alca

  • for the record

@cmsbuild
Copy link
Contributor

This issue is fully signed and ready to be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants