Skip to content

[BuildRules] Improved build rules for generating edm plugin cache - #10843

Merged
smuzaffar merged 2 commits into
IB/CMSSW_20_1_X/masterfrom
br-edmPluginRefresh
Sep 11, 2026
Merged

[BuildRules] Improved build rules for generating edm plugin cache#10843
smuzaffar merged 2 commits into
IB/CMSSW_20_1_X/masterfrom
br-edmPluginRefresh

Conversation

@smuzaffar

Copy link
Copy Markdown
Contributor

backport of #10833

it also contains changes from #10839

@smuzaffar

Copy link
Copy Markdown
Contributor Author

please test using full cmssw

@cmsbuild

cmsbuild commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

A new Pull Request was created by @smuzaffar for branch IB/CMSSW_20_1_X/master.

@akritkbehera, @iarspider, @raoatifshad, @smuzaffar can you please review it and eventually sign? Thanks.
@ftenchini, @mandrenguyen, @sextonkennedy you are the release manager for this.
cms-bot commands are listed here

@cmsbuild

cmsbuild commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

cms-bot internal usage

@cmsbuild

cmsbuild commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-a6b1e0/55775/summary.html
COMMIT: 11bda56
CMSSW: CMSSW_20_1_X_2026-09-09-1100/el9_amd64_gcc14
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmsdist/10843/55775/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

@smuzaffar

Copy link
Copy Markdown
Contributor Author

please test using full cmssw

@cmsbuild

Copy link
Copy Markdown
Contributor

Pull request #10843 was updated.

@cmsbuild

Copy link
Copy Markdown
Contributor

-1

Failed Tests: UnitTests
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-a6b1e0/55781/summary.html
COMMIT: ce594a9
CMSSW: CMSSW_20_1_X_2026-09-09-2300/el9_amd64_gcc14
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmsdist/10843/55781/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-a6b1e0/55781/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-a6b1e0/55781/git-merge-result

Failed Unit Tests

I found 10 errors in the following unit tests:

---> test CondToolsLHCInfoAnalyzerTest had ERRORS
---> test CondToolsPrintLHCInfoPerPayloads had ERRORS
---> test testmat had ERRORS
and more ...

Comparison Summary

Summary:

Max Memory Comparisons exceeding threshold

@cms-sw/core-l2 , I found 8 workflow step(s) with memory usage exceeding the error threshold:

Expand to see workflows ...
  • Error: Workflow 1001.0_RunMinBias2011A step8 max memory diff -83.6 exceeds +/- 30.0 MiB
  • Error: Workflow 1001.0_RunMinBias2011A step3 max memory diff -84.9 exceeds +/- 30.0 MiB
  • Error: Workflow 2025.0000002_RunZeroBias2025B_10k step3 max memory diff -680.8 exceeds +/- 30.0 MiB
  • Error: Workflow 2025.0000002_RunZeroBias2025B_10k step2 max memory diff -676.4 exceeds +/- 30.0 MiB
  • Error: Workflow 2025.0010001_RunJetMET02025C_10k step2 max memory diff -676.3 exceeds +/- 30.0 MiB
  • Error: Workflow 2025.0010001_RunJetMET02025C_10k step3 max memory diff -645.7 exceeds +/- 30.0 MiB
  • Error: Workflow 18434.0_TTbar_14TeV+2026 step2 max memory diff -675.5 exceeds +/- 30.0 MiB
  • Error: Workflow 18434.0_TTbar_14TeV+2026 step3 max memory diff -640.7 exceeds +/- 30.0 MiB

@smuzaffar

Copy link
Copy Markdown
Contributor Author

+externals

new build rules backported from DEVEL IBs

@smuzaffar
smuzaffar merged commit fa77cb2 into IB/CMSSW_20_1_X/master Sep 11, 2026
7 of 8 checks passed
@cmsbuild

Copy link
Copy Markdown
Contributor

This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_20_1_X/master IBs (but tests are reportedly failing). This pull request will now be reviewed by the release team before it's merged. @ftenchini, @sextonkennedy, @mandrenguyen (and backports should be raised in the release meeting by the corresponding L2)

@smuzaffar
smuzaffar deleted the br-edmPluginRefresh branch September 11, 2026 14:21
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.

2 participants