cmssw icon indicating copy to clipboard operation
cmssw copied to clipboard

[15_0_X] Muon skims for the 2025 light-ion runs

Open flodamas opened this issue 6 months ago • 9 comments

PR description:

Backport of #48386 for upcoming data taking.

PR validation:

dedicated runTheMatrix.py -w "standard" -l 143.921 completed successfully.

flodamas avatar Jun 24 '25 19:06 flodamas

A new Pull Request was created by @flodamas for CMSSW_15_0_X.

It involves the following packages:

  • Configuration/PyReleaseValidation (upgrade, pdmv)
  • Configuration/Skimming (pdmv)
  • Configuration/StandardSequences (operations)

@AdrianoDee, @Moanwar, @antoniovilela, @cmsbuild, @davidlange6, @dickychant, @fabiocos, @mandrenguyen, @miquork, @rappoccio, @srimanob, @subirsarkar can you please review it and eventually sign? Thanks. @GiacomoSguazzoni, @Martin-Grunewald, @VinInn, @VourMa, @denizsun, @dgulhan, @fabiocos, @felicepantaleo, @makortel, @missirol, @mmusich, @mtosi, @rovere, @salimcerci, @sameasy, @slomeo, @youyingli 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 #48386

cmsbuild avatar Jun 24 '25 19:06 cmsbuild

cms-bot internal usage

cmsbuild avatar Jun 24 '25 19:06 cmsbuild

Pull request #48397 was updated. @AdrianoDee, @Moanwar, @antoniovilela, @cmsbuild, @davidlange6, @dickychant, @fabiocos, @mandrenguyen, @miquork, @rappoccio, @srimanob, @subirsarkar can you please check and sign again.

cmsbuild avatar Jun 25 '25 18:06 cmsbuild

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

mandrenguyen avatar Jun 25 '25 18:06 mandrenguyen

please test workflow 143.911, 1000

mandrenguyen avatar Jun 26 '25 04:06 mandrenguyen

+1

Size: This PR adds an extra 96KB to repository Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9e0ac9/46927/summary.html COMMIT: 4b59f2107b86596f3fd86fdae93c8aa64a2361a6 CMSSW: CMSSW_15_0_X_2025-06-25-2300/el8_amd64_gcc12 User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/48397/46927/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

The workflows 143.911 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:

  • You potentially removed 14 lines from the logs
  • Reco comparison results: 2915 differences found in the comparisons
  • DQMHistoTests: Total files compared: 51
  • DQMHistoTests: Total histograms compared: 4057034
  • DQMHistoTests: Total failures: 10646
  • DQMHistoTests: Total nulls: 23
  • DQMHistoTests: Total successes: 4046345
  • DQMHistoTests: Total skipped: 20
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 30.213 KiB( 50 files compared)
  • DQMHistoSizes: changed ( 143.911 ): 29.305 KiB Hcal/DigiRunHarvesting
  • DQMHistoSizes: changed ( 143.911 ): 0.727 KiB RPC/DCSInfo
  • DQMHistoSizes: changed ( 143.911 ): 0.182 KiB SiStrip/MechanicalView
  • Checked 221 log files, 191 edm output root files, 51 DQM output files
  • TriggerResults: no differences found

cmsbuild avatar Jun 26 '25 08:06 cmsbuild

+pdmv

AdrianoDee avatar Jun 26 '25 12:06 AdrianoDee

+Upgrade

Moanwar avatar Jun 26 '25 12:06 Moanwar

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

cmsbuild avatar Jun 26 '25 12:06 cmsbuild

hold See: https://github.com/cms-sw/cmssw/pull/48386#issuecomment-3011803977

mandrenguyen avatar Jun 27 '25 06:06 mandrenguyen

Pull request has been put on hold by @mandrenguyen They need to issue an unhold command to remove the hold state or L1 can unhold it for all

cmsbuild avatar Jun 27 '25 06:06 cmsbuild

Pull request #48397 was updated. @AdrianoDee, @Moanwar, @antoniovilela, @cmsbuild, @davidlange6, @dickychant, @fabiocos, @mandrenguyen, @miquork, @rappoccio, @srimanob, @subirsarkar can you please check and sign again.

cmsbuild avatar Jun 27 '25 10:06 cmsbuild

unhold

mandrenguyen avatar Jun 27 '25 11:06 mandrenguyen

please test workflow 4.17,4.29

mmusich avatar Jun 27 '25 12:06 mmusich

+1

Size: This PR adds an extra 68KB to repository Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9e0ac9/46958/summary.html COMMIT: 0b1a240f32d00ae1e8238b68e5bb5c72d62896cd CMSSW: CMSSW_15_0_X_2025-06-27-1100/el8_amd64_gcc12 User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/48397/46958/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

cmsbuild avatar Jun 27 '25 15:06 cmsbuild

urgent @flodamas Please update the PR description to show that's a combined backport of 2 PRs. @cms-sw/pdmv-l2 @cms-sw/upgrade-l2 Please also have a look at this one.

mandrenguyen avatar Jun 27 '25 15:06 mandrenguyen

+pdmv

AdrianoDee avatar Jun 27 '25 15:06 AdrianoDee

+Upgrade

Moanwar avatar Jun 27 '25 16:06 Moanwar

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

cmsbuild avatar Jun 27 '25 16:06 cmsbuild

+1

mandrenguyen avatar Jun 28 '25 06:06 mandrenguyen