[14_0_X] Backports of 46543 and 46451 to fix memory leak

[srimanob]

PR description:

Try to fix memory leak issue, mentionedin #46901, for ReReco 2024 with CMSSW_14_0.

Backports of

• make AlcaBeamMonitor hoard less memory #46451
• Fix memory leak issues in Calo Layer 1 UCT Infrastructure #46543

PR validation:

• CMSSW with this PR can run.

If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:

[cmsbuild]

A new Pull Request was created by @srimanob for CMSSW_14_0_X.

It involves the following packages:

• DQM/BeamMonitor (db, dqm)
• L1Trigger/L1TCaloLayer1 (l1)

@aloeliger, @antoniovagnerini, @atpathak, @cmsbuild, @consuegs, @epalencia, @francescobrivio, @perrotta, @rseidita can you please review it and eventually sign? Thanks.
@JanChyczynski, @Martin-Grunewald, @PonIlya, @francescobrivio, @missirol, @mmusich, @rsreds, @yuanchao this is something you requested to watch as well.
@antoniovilela, @mandrenguyen, @rappoccio, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

• Backported from make AlcaBeamMonitor hoard less memory #46451

[cmsbuild]

cms-bot internal usage

[srimanob]

@cmsbuild please test

[srimanob]

This is an effort to handle #46901

[cmsbuild]

+1

Size: This PR adds an extra 16KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-cd0061/43340/summary.html
COMMIT: e4453ac
CMSSW: CMSSW_14_0_X_2024-12-09-1100/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/46903/43340/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• You potentially removed 3 lines from the logs
• Reco comparison results: 44 differences found in the comparisons
• DQMHistoTests: Total files compared: 49
• DQMHistoTests: Total histograms compared: 3456543
• DQMHistoTests: Total failures: 394
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3456129
• DQMHistoTests: Total skipped: 20
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 48 files compared)
• Checked 206 log files, 170 edm output root files, 49 DQM output files
• TriggerResults: no differences found

[antoniovagnerini]

+dqm

[perrotta]

+db

[srimanob]

Kindly ping @cms-sw/l1-l2 in case we need to build the new release for reprocessing. Thanks.

[srimanob]

@cms-sw/orp-l2 Could you please consider this backport? We will need a new release for ReReco.

FYI @vlimant @malbouis

[srimanob]

The comparison between with and without this PR can be found at #46901 (comment)

[aloeliger]

+l1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_14_0_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_15_0_X is complete. This pull request will now be reviewed by the release team before it's merged. @mandrenguyen, @sextonkennedy, @antoniovilela, @rappoccio (and backports should be raised in the release meeting by the corresponding L2)

[mandrenguyen]

@srimanob should we consider backporting to intermediate releases?

[srimanob]

@srimanob should we consider backporting to intermediate releases?

Hi Matt, Do you mean 14_1? Yes, I can do it. Thx.
I first need it for 14_0 used for ReReco.

[mandrenguyen]

@srimanob should we consider backporting to intermediate releases?

Hi Matt, Do you mean 14_1? Yes, I can do it. Thx. I first need it for 14_0 used for ReReco.

Yes, please make the PRs to the intermediate release(s) and then we'll merge this.

[srimanob]

Hi @mandrenguyen
Both backport PRs were merged in 14_1:
#46465 ([14.1.X] make AlcaBeamMonitor hoard less memory)
#46567 ([14_1] Fix memory leak issues in Calo Layer 1 UCT Infrastructure)

[mandrenguyen]

backport of #46451

[mandrenguyen]

+1

[mandrenguyen]

merge

[iarspider]

@srimanob could you please resolve conflicts? Thanks!

[srimanob]

Hi @iarspider
How do show which conflicts? I just see the warning shows up.

[smuzaffar]

Looks like some issue with github itself. PR has been merged. Let me close this

[smuzaffar]

bot can see that this has been merged. It is visible as merged in IB page https://cmssdt.cern.ch/SDT/html/cmssdt-ib/#/ib/CMSSW_14_0_X