Skip to content

[EVOLUTION_X] Update checksums for MtdSim{LayerCluster,Trackster}#50485

Merged
cmsbuild merged 1 commit into
cms-sw:CMSSW_16_1_EVOLUTION_Xfrom
makortel:patch-9
Mar 23, 2026
Merged

[EVOLUTION_X] Update checksums for MtdSim{LayerCluster,Trackster}#50485
cmsbuild merged 1 commit into
cms-sw:CMSSW_16_1_EVOLUTION_Xfrom
makortel:patch-9

Conversation

@makortel
Copy link
Copy Markdown
Contributor

@makortel makortel commented Mar 20, 2026

PR description:

Knock-on effect of #50418, should fix #50483 (comment).

Resolves cms-sw/framework-team#2066

PR validation:

None

@cmsbuild
Copy link
Copy Markdown
Contributor

cmsbuild commented Mar 20, 2026

cms-bot internal usage

@cmsbuild
Copy link
Copy Markdown
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50485/48637

@cmsbuild
Copy link
Copy Markdown
Contributor

A new Pull Request was created by @makortel for CMSSW_16_1_EVOLUTION_X.

It involves the following packages:

  • SimDataFormats/CaloAnalysis (simulation)

@civanch, @cmsbuild, @kpedro88, @mdhildreth can you please review it and eventually sign? Thanks.
@ReyerBand, @apsallid, @bsunanda, @denizsun, @fabiocos, @felicepantaleo, @martinamalberti, @missirol, @mmusich, @rovere, @salimcerci, @thomreis, @wang0jin this is something you requested to watch as well.
@ftenchini, @mandrenguyen, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

@makortel
Copy link
Copy Markdown
Contributor Author

@cmsbuild, please test

@cmsbuild
Copy link
Copy Markdown
Contributor

-1

Failed Tests: UnitTests RelVals AddOn
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-83fe10/52127/summary.html
COMMIT: 4e7ffb8
CMSSW: CMSSW_16_1_EVOLUTION_X_2026-03-20-2300/el8_amd64_gcc13
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/50485/52127/install.sh to create a dev area with all the needed externals and cmssw changes.

Failed Unit Tests

I found 2 errors in the following unit tests:

---> test test_MC_22_crosscheck had ERRORS
---> test test_MC_23_crosscheck had ERRORS

Failed RelVals

  • 135.4135.4_ZEEFS_13/step1_ZEEFS_13.log
  • 1000.01000.0_RunMinBias2011A/step2_RunMinBias2011A.log
  • 2025.00000012025.0000001_RunZeroBias2025B_10k/step2_RunZeroBias2025B_10k.log
Expand to see more relval errors ...

Failed AddOn Tests

UNKNOWN
UNKNOWN
UNKNOWN
Expand to see more addon errors ...

@makortel
Copy link
Copy Markdown
Contributor Author

The failures look the same as in the IB, that is broken because the changes of this PR are not in. The PR tests did not appear to check out the packages depending on SimDataFormats/CaloAnalysis (probably because the changes were only in classes_def.xml?), and because of that the link errors of the IB were not fixed in the tests of this PR.

@makortel
Copy link
Copy Markdown
Contributor Author

ignore tests-rejected with manual override

@makortel
Copy link
Copy Markdown
Contributor Author

assign core

@cmsbuild
Copy link
Copy Markdown
Contributor

New categories assigned: core

@Dr15Jones,@makortel,@smuzaffar you have been requested to review this Pull request/Issue and eventually sign? Thanks

@makortel
Copy link
Copy Markdown
Contributor Author

type evolution

@makortel
Copy link
Copy Markdown
Contributor Author

+core

@Dr15Jones
Copy link
Copy Markdown
Contributor

ignore tests-rejected with manual-override

@Dr15Jones
Copy link
Copy Markdown
Contributor

@smuzaffar it appears that ignore tests-rejected with manual-override is not causing the tag test-rejected to be replaced

@smuzaffar
Copy link
Copy Markdown
Contributor

@smuzaffar it appears that ignore tests-rejected with manual-override is not causing the tag test-rejected to be replaced

@Dr15Jones , looks like github failed to deliver the initial webhook to bot that is why bot was not able to process you initial comment but your 2nd comment triiger the bot and it correctly updated the labels

@mandrenguyen
Copy link
Copy Markdown
Contributor

merge

@cmsbuild cmsbuild merged commit ba0c3d1 into cms-sw:CMSSW_16_1_EVOLUTION_X Mar 23, 2026
6 of 9 checks passed
@makortel makortel deleted the patch-9 branch March 23, 2026 13:43
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.

5 participants