use automatic upgrade Detector number redirection introduced in #49437 in the GPU matrix by mmusich · Pull Request #49647 · cms-sw/cmssw

mmusich · 2025-12-16T18:00:03Z

PR description:

Title says it all, I noticed that the GPU matrix uses a different detector geometry (D110) in lieu of the one used for the rest of the workflows (D121, introduced in #49437).
I propose to use the same mechanism also in the GPU matrix.

PR validation:

None, to be done in this PR.

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:

N/A

…in the GPU matrix

cmsbuild · 2025-12-16T18:00:29Z

cms-bot internal usage

mmusich · 2025-12-16T18:00:50Z

test parameters:

pull_request = update phase-2 GPU workflow to use D121 cms-bot#2636

mmusich · 2025-12-16T18:00:56Z

enable gpu

cmsbuild · 2025-12-16T18:01:28Z

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49647/47189

cmsbuild · 2025-12-16T18:01:48Z

A new Pull Request was created by @mmusich for master.

It involves the following packages:

Configuration/PyReleaseValidation (pdmv)

@AdrianoDee, @DickyChant, @antoniovagnerini, @cmsbuild, @miquork can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @fabiocos, @makortel, @slomeo 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

mmusich · 2025-12-16T18:02:34Z

@cmsbuild, please test

AdrianoDee · 2025-12-16T19:10:05Z

I suspect you'd need to keep the 29834.402,29834.403,29834.404,29834.704,29834.751 wfs used by the bot and hardcoded for this to work a first time, and then change those there and here again (at least this was the workaround I had found) .

mmusich · 2025-12-16T19:11:29Z

I suspect you'd need to keep the 29834.402,29834.403,29834.404,29834.704,29834.751 wfs used by the bot and hardcoded for this to work a first time, and then change those there and here again (at least this was the workaround I had found) .

isn't this what is proposed at cms-sw/cms-bot#2636 ?

AdrianoDee · 2025-12-16T19:14:16Z

Oh, right, I had missed it, my bad!

cmsbuild · 2025-12-17T22:49:09Z

+1

Size: This PR adds an extra 20KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-bb350a/50005/summary.html
COMMIT: a259344
CMSSW: CMSSW_16_0_X_2025-12-16-1100/el8_amd64_gcc13
Additional Tests: GPU,AMD_MI300X,AMD_W7900,NVIDIA_H100,NVIDIA_L40S,NVIDIA_T4
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/49647/50005/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

You potentially removed 1 lines from the logs
ROOTFileChecks: Some differences in event products or their sizes found
Reco comparison results: 4 differences found in the comparisons
Reco comparison had 4 failed jobs
DQMHistoTests: Total files compared: 53
DQMHistoTests: Total histograms compared: 4280229
DQMHistoTests: Total failures: 44
DQMHistoTests: Total nulls: 0
DQMHistoTests: Total successes: 4280165
DQMHistoTests: Total skipped: 20
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 0.0 KiB( 52 files compared)
Checked 227 log files, 198 edm output root files, 53 DQM output files
TriggerResults: no differences found

AMD_MI300X Comparison Summary

Summary:

No significant changes to the logs found
Reco comparison results: 0 differences found in the comparisons
DQMHistoTests: Total files compared: 1
DQMHistoTests: Total histograms compared: 0
DQMHistoTests: Total failures: 0
DQMHistoTests: Total nulls: 0
DQMHistoTests: Total successes: 0
DQMHistoTests: Total skipped: 0
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 0 KiB( 0 files compared)
Checked 0 log files, 0 edm output root files, 1 DQM output files

AMD_W7900 Comparison Summary

Summary:

No significant changes to the logs found
Reco comparison results: 0 differences found in the comparisons
DQMHistoTests: Total files compared: 1
DQMHistoTests: Total histograms compared: 0
DQMHistoTests: Total failures: 0
DQMHistoTests: Total nulls: 0
DQMHistoTests: Total successes: 0
DQMHistoTests: Total skipped: 0
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 0 KiB( 0 files compared)
Checked 0 log files, 0 edm output root files, 1 DQM output files

NVIDIA_H100 Comparison Summary

Summary:

No significant changes to the logs found
Reco comparison results: 0 differences found in the comparisons
DQMHistoTests: Total files compared: 1
DQMHistoTests: Total histograms compared: 0
DQMHistoTests: Total failures: 0
DQMHistoTests: Total nulls: 0
DQMHistoTests: Total successes: 0
DQMHistoTests: Total skipped: 0
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 0 KiB( 0 files compared)
Checked 0 log files, 0 edm output root files, 1 DQM output files

NVIDIA_L40S Comparison Summary

Summary:

No significant changes to the logs found
Reco comparison results: 0 differences found in the comparisons
DQMHistoTests: Total files compared: 1
DQMHistoTests: Total histograms compared: 0
DQMHistoTests: Total failures: 0
DQMHistoTests: Total nulls: 0
DQMHistoTests: Total successes: 0
DQMHistoTests: Total skipped: 0
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 0 KiB( 0 files compared)
Checked 0 log files, 0 edm output root files, 1 DQM output files

NVIDIA_T4 Comparison Summary

Summary:

You potentially added 1 lines to the logs
Reco comparison results: 63 differences found in the comparisons
Reco comparison had 6 failed jobs
DQMHistoTests: Total files compared: 7
DQMHistoTests: Total histograms compared: 55076
DQMHistoTests: Total failures: 9061
DQMHistoTests: Total nulls: 0
DQMHistoTests: Total successes: 46015
DQMHistoTests: Total skipped: 0
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 0.0 KiB( 6 files compared)
Checked 24 log files, 30 edm output root files, 7 DQM output files
TriggerResults: no differences found

mmusich · 2025-12-18T08:32:29Z

isn't this what is proposed at cms-sw/cms-bot#2636 ?

by looking here this seems to have worked!

AdrianoDee · 2025-12-18T08:37:46Z

+pdmv

cmsbuild · 2025-12-18T08:38:06Z

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @sextonkennedy, @ftenchini, @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/cms-bot#2636

cmsbuild · 2025-12-18T13:00:40Z

Milestone for this pull request has been moved to CMSSW_16_1_X. Please open a backport if it should also go in to CMSSW_16_0_X.

ftenchini · 2025-12-22T09:19:09Z

+1

makortel · 2025-12-22T19:30:28Z

Just to note that merging this PR without cms-sw/cms-bot#2636 is causing GPU-enabled PR tests to fail, see #49700.

use automatic Detector number redirection introduced in cms-sw#49437 …

a259344

…in the GPU matrix

mmusich mentioned this pull request Dec 16, 2025

update phase-2 GPU workflow to use D121 cms-sw/cms-bot#2636

Merged

cmsbuild added this to the CMSSW_16_0_X milestone Dec 16, 2025

cmsbuild added pending-signatures tests-pending orp-pending pdmv-pending code-checks-pending labels Dec 16, 2025

cmsbuild added the requires-external label Dec 16, 2025

cmsbuild added code-checks-approved and removed code-checks-pending labels Dec 16, 2025

cmsbuild added tests-started and removed tests-pending labels Dec 16, 2025

cmsbuild added tests-approved and removed tests-started labels Dec 17, 2025

cmsbuild added fully-signed and removed pending-signatures pdmv-pending labels Dec 18, 2025

cmsbuild added the pdmv-approved label Dec 18, 2025

cmsbuild modified the milestones: CMSSW_16_0_X, CMSSW_16_1_X Dec 18, 2025

cmsbuild added orp-approved and removed orp-pending labels Dec 22, 2025

cmsbuild merged commit ebeaa76 into cms-sw:master Dec 22, 2025
26 checks passed

mmusich deleted the mm_gpu_matrix_update branch December 22, 2025 13:33

makortel mentioned this pull request Dec 22, 2025

GPU PR tests failing with "Undefined workflows" #49700

Closed

Conversation

mmusich commented Dec 16, 2025

PR description:

PR validation:

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:

Uh oh!

cmsbuild commented Dec 16, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

mmusich commented Dec 16, 2025

Uh oh!

mmusich commented Dec 16, 2025

Uh oh!

cmsbuild commented Dec 16, 2025

Uh oh!

cmsbuild commented Dec 16, 2025

Uh oh!

mmusich commented Dec 16, 2025

Uh oh!

AdrianoDee commented Dec 16, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

mmusich commented Dec 16, 2025

Uh oh!

AdrianoDee commented Dec 16, 2025

Uh oh!

cmsbuild commented Dec 17, 2025

Comparison Summary

AMD_MI300X Comparison Summary

AMD_W7900 Comparison Summary

NVIDIA_H100 Comparison Summary

NVIDIA_L40S Comparison Summary

NVIDIA_T4 Comparison Summary

Uh oh!

mmusich commented Dec 18, 2025

Uh oh!

AdrianoDee commented Dec 18, 2025

Uh oh!

cmsbuild commented Dec 18, 2025

Uh oh!

cmsbuild commented Dec 18, 2025

Uh oh!

ftenchini commented Dec 22, 2025

Uh oh!

Uh oh!

makortel commented Dec 22, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

cmsbuild commented Dec 16, 2025 •

edited

Loading

AdrianoDee commented Dec 16, 2025 •

edited

Loading