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

adding new pdl for bc in 7_1_x #4101

Merged
merged 2 commits into from Jun 26, 2018

Conversation

alberto-sanchez
Copy link
Member

No description provided.

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 8, 2018

A new Pull Request was created by @alberto-sanchez (Alberto Sanchez Hernandez) for branch IB/CMSSW_7_1_X/stable.

@cmsbuild, @smuzaffar, @gudrutis, @mrodozov can you please review it and eventually sign? Thanks.
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.

@smuzaffar
Copy link
Contributor

backport of #4069

@smuzaffar
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 8, 2018

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

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 8, 2018

-1

Tested at: 0031fd5

  • Build:

I found an error when building:

+ case "${DOWNLOAD_TOOL}" in
+ touch V02-00-10.tar.gz
+ curl -L -O -k https://github.com/cms-data/GeneratorInterface-EvtGenInterface/archive/V02-00-10.tar.gz

curl: (35) SSL connect error
error: Bad exit status from /build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/tmp/rpm-tmp.L1WhrY (%build)


RPM build errors:
Bad exit status from /build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/tmp/rpm-tmp.L1WhrY (%build)



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

@alberto-sanchez
Copy link
Member Author

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 9, 2018

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

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 9, 2018

-1

Tested at: 0031fd5

  • Build:

I found an error when building:

+ case "${DOWNLOAD_TOOL}" in
+ touch V02-00-10.tar.gz
+ curl -L -O -k https://github.com/cms-data/GeneratorInterface-EvtGenInterface/archive/V02-00-10.tar.gz

curl: (35) SSL connect error
error: Bad exit status from /build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/tmp/rpm-tmp.2rzuLA (%build)


RPM build errors:
Bad exit status from /build/cmsbld/jenkins/workspace/ib-any-integration/testBuildDir/tmp/rpm-tmp.2rzuLA (%build)



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

@alberto-sanchez
Copy link
Member Author

Does this has to do with the depreciation of https connection to github?

@smuzaffar
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 9, 2018

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

@alberto-sanchez
Copy link
Member Author

@fabiocos the MC request which uses this table has been requested. Could you please consider to merge this PR and make a new release?

@fabiocos
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 22, 2018

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

@cmsbuild
Copy link
Contributor

-1

Tested at: de2832c

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

I found follow errors while testing this PR

Failed tests: AddOn

  • AddOn:

I found errors in the following addon tests:

cmsRun /cvmfs/cms-ib.cern.ch/week1/slc6_amd64_gcc481/cms/cmssw/CMSSW_7_1_X_2018-06-17-0000/src/PhysicsTools/PatAlgos/test/IntegrationTest_cfg.py : FAILED - time: date Fri Jun 22 14:41:48 2018-date Fri Jun 22 14:36:28 2018 s - exit: 16640

@fabiocos
Copy link
Contributor

The failing AddOn test has nothing to do with Generators:

----- Begin Fatal Exception 22-Jun-2018 14:41:46 CEST-----------------------
An exception of category 'DDException' occurred while
[0] Processing run: 1 lumi: 1 event: 1
[1] Running path 'p'
[2] Calling event method for module PATElectronSelector/'selectedPatElectrons'
[3] Calling produce method for unscheduled module PATElectronProducer/'patElectrons'
[4] Using EventSetup component CaloTopologyBuilder/'' to make data CaloTopology/'' in record CaloTopologyRecord
[5] Using EventSetup component CaloGeometryBuilder/'' to make data CaloGeometry/'' in record CaloGeometryRecord
[6] Using EventSetup component EcalBarrelGeometryEP/'' to make data CaloSubdetectorGeometry/'EcalBarrel' in record EcalBarrelGeometryRecord
[7] Using EventSetup component XMLIdealGeometryESSource/'' to make data DDCompactView/'' in record IdealGeometryRecord
Exception Message:

DDLParser, algo requested is not defined.  Either AlgoInit() or check algo spelling.

	algo=global:angular
	parent=hcalalgo:HRCF
	self=hcalalgo:HEC1

Element AlgoPosPart
----- End Fatal Exception -------------------------------------------------

It looks funny that this failure did not appear in the recent IB

@cmsbuild
Copy link
Contributor

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

Comparison Summary:

  • No significant changes to the logs found
  • Reco comparison results: 0 differences found in the comparisons
  • DQMHistoTests: Total files compared: 11
  • DQMHistoTests: Total histograms compared: 682535
  • DQMHistoTests: Total failures: 2902
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 679259
  • DQMHistoTests: Total skipped: 374
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: -10 KiB( 10 files compared)
  • Checked 44 log files, 4 edm output root files, 11 DQM output files

@fabiocos
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 25, 2018

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

@cmsbuild
Copy link
Contributor

-1

Tested at: de2832c

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

I found follow errors while testing this PR

Failed tests: AddOn

  • AddOn:

I found errors in the following addon tests:

cmsRun /cvmfs/cms-ib.cern.ch/week0/slc6_amd64_gcc481/cms/cmssw-patch/CMSSW_7_1_X_2018-06-24-2300/src/PhysicsTools/PatAlgos/test/IntegrationTest_cfg.py : FAILED - time: date Mon Jun 25 11:15:26 2018-date Mon Jun 25 11:10:29 2018 s - exit: 16640

@cmsbuild
Copy link
Contributor

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

Comparison Summary:

  • No significant changes to the logs found
  • Reco comparison results: 0 differences found in the comparisons
  • DQMHistoTests: Total files compared: 11
  • DQMHistoTests: Total histograms compared: 682535
  • DQMHistoTests: Total failures: 3342
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 678819
  • DQMHistoTests: Total skipped: 374
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: -10 KiB( 10 files compared)
  • Checked 44 log files, 4 edm output root files, 11 DQM output files

@fabiocos
Copy link
Contributor

@smuzaffar I keep getting this AddOn test failure that is not observed in the base 7_1_X test, even running it in a standalone way. It anyway looks unrelated to the tested external change, as it has nothing to do with EvtGen. Any idea? Otherwise I would just merge it and see what happens in the IB.

@fabiocos
Copy link
Contributor

+1

@fabiocos
Copy link
Contributor

+externals

@fabiocos
Copy link
Contributor

@smuzaffar can we merge this for the next IB?

@fabiocos
Copy link
Contributor

merge

@cmsbuild cmsbuild merged commit db55dab into cms-sw:IB/CMSSW_7_1_X/stable Jun 26, 2018
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