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 ppjjjj library [10_6_X backport] #6966

Merged
merged 3 commits into from Jun 3, 2021

Conversation

SiewYan
Copy link
Contributor

@SiewYan SiewYan commented Jun 2, 2021

Adding missing library, needed for SMP's four-jet production study.

backport from
#6956

adding missing library ppjjjj
@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 2, 2021

A new Pull Request was created by @SiewYan (Siewyan Hoh 何守仁) for branch IB/CMSSW_10_6_X/gcc700.

@cmsbuild, @smuzaffar, @mrodozov, @iarspider can you please review it and eventually sign? Thanks.
cms-bot commands are listed here

@SiewYan
Copy link
Contributor Author

SiewYan commented Jun 2, 2021

please test

@SiewYan
Copy link
Contributor Author

SiewYan commented Jun 2, 2021

assign generators

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 2, 2021

New categories assigned: generators

@mkirsano,@SiewYan,@alberto-sanchez,@agrohsje,@GurpreetSinghChahal you have been requested to review this Pull request/Issue and eventually sign? Thanks

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 2, 2021

-1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-e37bac/15546/summary.html
COMMIT: 9050f8c
CMSSW: CMSSW_10_6_X_2021-06-02-1100/slc7_amd64_gcc700
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmsdist/6966/15546/install.sh to create a dev area with all the needed externals and cmssw changes.

External Build

I found compilation error when building:

- process: ppzjjj ... download from repository: public... extract ... done
- process: pphbb ... download from repository: public... extract ... done
- process: ppvvv2 ... download from repository: public... extract ... done
- process: heftpplljjj ... ERROR: not available (installed: {})
ERROR: process downloader failed.
error: Bad exit status from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/tmp/rpm-tmp.yYLs6R (%build)


RPM build errors:
Bad exit status from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/tmp/rpm-tmp.yYLs6R (%build)



@SiewYan
Copy link
Contributor Author

SiewYan commented Jun 2, 2021

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 2, 2021

Pull request #6966 was updated.

@mrodozov
Copy link
Contributor

mrodozov commented Jun 2, 2021

@SiewYan if you remember we had to remove files some time ago from 12_0 because they were not there anymore
I guess openloops for 10_6 didn't had to rebuild and thats why never failed

@SiewYan
Copy link
Contributor Author

SiewYan commented Jun 2, 2021

@mrodozov yeah! some of those libs are no longer available. Should I synchronize this list with 12_0_X's?

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 2, 2021

-1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-e37bac/15548/summary.html
COMMIT: c112077
CMSSW: CMSSW_10_6_X_2021-06-02-1100/slc7_amd64_gcc700
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmsdist/6966/15548/install.sh to create a dev area with all the needed externals and cmssw changes.

External Build

I found compilation error when building:

- process: ppzjjj ... download from repository: public... extract ... done
- process: pphbb ... download from repository: public... extract ... done
- process: ppvvv2 ... download from repository: public... extract ... done
- process: heftpplljjj ... ERROR: not available (installed: {})
ERROR: process downloader failed.
error: Bad exit status from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/tmp/rpm-tmp.3PR7YX (%build)


RPM build errors:
Bad exit status from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/tmp/rpm-tmp.3PR7YX (%build)



@mrodozov
Copy link
Contributor

mrodozov commented Jun 2, 2021

@SiewYan best do that, yes :)

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 3, 2021

Pull request #6966 was updated.

@SiewYan
Copy link
Contributor Author

SiewYan commented Jun 3, 2021

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 3, 2021

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-e37bac/15589/summary.html
COMMIT: 4f07758
CMSSW: CMSSW_10_6_X_2021-06-02-2300/slc7_amd64_gcc700
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmsdist/6966/15589/install.sh to create a dev area with all the needed externals and cmssw changes.

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:

You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-e37bac/15589/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-e37bac/15589/git-merge-result

Comparison Summary

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

Summary:

  • No significant changes to the logs found
  • Reco comparison results: 1233 differences found in the comparisons
  • DQMHistoTests: Total files compared: 35
  • DQMHistoTests: Total histograms compared: 3215543
  • DQMHistoTests: Total failures: 2041
  • DQMHistoTests: Total nulls: 22
  • DQMHistoTests: Total successes: 3213146
  • DQMHistoTests: Total skipped: 334
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: -56.705 KiB( 34 files compared)
  • DQMHistoSizes: changed ( 140.53 ): -44.531 KiB Hcal/DigiRunHarvesting
  • DQMHistoSizes: changed ( 140.53 ): -10.938 KiB Info/EventInfo
  • DQMHistoSizes: changed ( 140.53 ): -1.172 KiB RPC/DCSInfo
  • DQMHistoSizes: changed ( 140.53 ): -0.064 KiB SiStrip/MechanicalView
  • Checked 143 log files, 29 edm output root files, 35 DQM output files
  • TriggerResults: no differences found

@mrodozov
Copy link
Contributor

mrodozov commented Jun 3, 2021

+externals

@agrohsje
Copy link

agrohsje commented Jun 3, 2021

+generators

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 3, 2021

This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_10_6_X/gcc700 IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2)

@qliphy
Copy link
Contributor

qliphy commented Jun 3, 2021

+1

@cmsbuild cmsbuild merged commit e9b71dc into cms-sw:IB/CMSSW_10_6_X/gcc700 Jun 3, 2021
@SiewYan SiewYan deleted the patch-1 branch June 3, 2021 13:38
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