Skip to content

fix finding old externals size/file count#2810

Merged
smuzaffar merged 1 commit into
masterfrom
fix-ext-stats
Jul 9, 2026
Merged

fix finding old externals size/file count#2810
smuzaffar merged 1 commit into
masterfrom
fix-ext-stats

Conversation

@smuzaffar

Copy link
Copy Markdown
Contributor

changes are needed due to update in toolfile format in 17.0.X and above IBs

@smuzaffar

Copy link
Copy Markdown
Contributor Author

please test with cms-sw/cmsdist#10711

@cmsbuild

cmsbuild commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

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

@akritkbehera, @cmsbuild, @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 Jul 9, 2026

Copy link
Copy Markdown
Contributor

cms-bot internal usage

@smuzaffar

Copy link
Copy Markdown
Contributor Author

+externals

@smuzaffar smuzaffar merged commit 8d9cf31 into master Jul 9, 2026
4 of 5 checks passed
@cmsbuild

cmsbuild commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

This pull request is fully signed and it will be integrated in one of the next master IBs after it passes the integration tests. 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)
Notice This PR was tested with additional Pull Request(s), please also merge them if necessary: cms-sw/cmsdist#10711

@cmsbuild

cmsbuild commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b88c5a/54629/summary.html
COMMIT: ef0c214
CMSSW: CMSSW_20_1_X_2026-07-08-2300/el9_amd64_gcc13
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cms-bot/2810/54629/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

  • You potentially removed 42 lines from the logs
  • Reco comparison results: 44240 differences found in the comparisons
  • DQMHistoTests: Total files compared: 45
  • DQMHistoTests: Total histograms compared: 3548186
  • DQMHistoTests: Total failures: 420798
  • DQMHistoTests: Total nulls: 294
  • DQMHistoTests: Total successes: 3127076
  • DQMHistoTests: Total skipped: 18
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 5.768 KiB( 44 files compared)
  • DQMHistoSizes: changed ( 7.3 ): 5.768 KiB SiStrip/MechanicalView
  • Checked 195 log files, 163 edm output root files, 45 DQM output files
  • TriggerResults: found differences in 10 / 44 workflows

Max Memory Comparisons exceeding threshold

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

Expand to see workflows ...
  • Error: Workflow 18434.0_TTbar_14TeV+2026 step5 max memory diff 33.2 exceeds +/- 30.0 MiB
  • Error: Workflow 34434.911_TTbar_14TeV+Run4D121_DD4hep step5 max memory diff 57.5 exceeds +/- 30.0 MiB
  • Error: Workflow 34634.0_TTbar_14TeV+Run4D121PU step2 max memory diff -263.4 exceeds +/- 30.0 MiB
  • Error: Workflow 34634.0_TTbar_14TeV+Run4D121PU step3 max memory diff -671.5 exceeds +/- 30.0 MiB

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