cmssw icon indicating copy to clipboard operation
cmssw copied to clipboard

[14_0_X] Fix for is not a valid DT id error

Open jfernan2 opened this issue 11 months ago • 3 comments

PR description:

Verbatim backport of https://github.com/cms-sw/cmssw/pull/44417

jfernan2 avatar Mar 15 '24 12:03 jfernan2

type bug-fix

jfernan2 avatar Mar 15 '24 12:03 jfernan2

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

It involves the following packages:

  • L1Trigger/DTTriggerPhase2 (l1, upgrade)

@srimanob, @aloeliger, @epalencia, @cmsbuild, @subirsarkar can you please review it and eventually sign? Thanks. @missirol, @Martin-Grunewald, @battibass, @JanFSchulte this is something you requested to watch as well. @antoniovilela, @rappoccio, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

  • Backported from #44417

cmsbuild avatar Mar 15 '24 12:03 cmsbuild

cms-bot internal usage

cmsbuild avatar Mar 15 '24 12:03 cmsbuild

backport of https://github.com/cms-sw/cmssw/pull/44417

aloeliger avatar Mar 18 '24 14:03 aloeliger

please test

aloeliger avatar Mar 25 '24 12:03 aloeliger

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5c8508/38387/summary.html COMMIT: 43d1393b42ca9589769c6871fbae49b9775563a3 CMSSW: CMSSW_14_0_X_2024-03-25-1100/el8_amd64_gcc12 User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/44418/38387/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

  • You potentially removed 164 lines from the logs
  • ROOTFileChecks: Some differences in event products or their sizes found
  • Reco comparison results: 29 differences found in the comparisons
  • DQMHistoTests: Total files compared: 48
  • DQMHistoTests: Total histograms compared: 3243082
  • DQMHistoTests: Total failures: 0
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 3243060
  • DQMHistoTests: Total skipped: 22
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 47 files compared)
  • Checked 202 log files, 163 edm output root files, 48 DQM output files
  • TriggerResults: no differences found

cmsbuild avatar Mar 25 '24 20:03 cmsbuild

+l1

aloeliger avatar Mar 26 '24 11:03 aloeliger

+Upgrade

srimanob avatar Mar 26 '24 11:03 srimanob

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_14_1_X is complete. This pull request will now be reviewed by the release team before it's merged. @antoniovilela, @rappoccio, @sextonkennedy (and backports should be raised in the release meeting by the corresponding L2)

cmsbuild avatar Mar 26 '24 11:03 cmsbuild

+1

antoniovilela avatar Mar 27 '24 19:03 antoniovilela