Skip to content

Fixed D127 Inputs - #51717

Merged
cmsbuild merged 1 commit into
cms-sw:masterfrom
AdrianoDee:new_d127_inputs_fix
Aug 17, 2026
Merged

Fixed D127 Inputs#51717
cmsbuild merged 1 commit into
cms-sw:masterfrom
AdrianoDee:new_d127_inputs_fix

Conversation

@AdrianoDee

@AdrianoDee AdrianoDee commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

PR description:

A follow-up to #51684 is needed because some samples (namely the TTbar ones) have a different v* in the dataset name due to the fact that the samples had to be resubmitted. In the resubmission process, the first partially completed samples have been stored and then invalidated. In the time between these two operations, the tests have run successfully, not showing this issue.

The different v (v2) makes them "invisible".

Long story short we were obliged to reproduce new GEN-SIM samples as inputs.

This change is propagated to the [20_0_X] PR #51685.

@cmsbuild

cmsbuild commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

cms-bot internal usage

@AdrianoDee

Copy link
Copy Markdown
Contributor Author

please test

@cmsbuild

Copy link
Copy Markdown
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-51717/50631

@cmsbuild

Copy link
Copy Markdown
Contributor

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

It involves the following packages:

  • Configuration/PyReleaseValidation (pdmv)

@AdrianoDee, @DickyChant, @antoniovagnerini, @kfjack, @miquork, @sroychow 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

@cmsbuild

Copy link
Copy Markdown
Contributor

+1

Size: This PR adds an extra 68KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-33cd90/55388/summary.html
COMMIT: d44053e
CMSSW: CMSSW_20_1_X_2026-08-16-0000/el9_amd64_gcc14
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/51717/55388/install.sh to create a dev area with all the needed externals and cmssw changes.

DAS Queries: The DAS query tests failed, see the summary page for details.

Comparison Summary

Summary:

  • You potentially removed 16 lines from the logs
  • ROOTFileChecks: Some differences in event products or their sizes found
  • Reco comparison results: 4196 differences found in the comparisons
  • DQMHistoTests: Total files compared: 45
  • DQMHistoTests: Total histograms compared: 3591807
  • DQMHistoTests: Total failures: 85280
  • DQMHistoTests: Total nulls: 160
  • DQMHistoTests: Total successes: 3506349
  • DQMHistoTests: Total skipped: 18
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 44 files compared)
  • Checked 195 log files, 163 edm output root files, 45 DQM output files
  • TriggerResults: found differences in 1 / 44 workflows

Max Memory Comparisons exceeding threshold

@cms-sw/core-l2 , I found 2 workflow step(s) with memory usage exceeding the error threshold:

Expand to see workflows ...
  • Error: Workflow 37834.0_TTbar_14TeV+Run4D127PU step3 max memory diff -700.5 exceeds +/- 30.0 MiB
  • Error: Workflow 37834.0_TTbar_14TeV+Run4D127PU step2 max memory diff -85.6 exceeds +/- 30.0 MiB

@AdrianoDee

Copy link
Copy Markdown
Contributor Author

+pdmv

@mandrenguyen

Copy link
Copy Markdown
Contributor

+1

@cmsbuild

Copy link
Copy Markdown
Contributor

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 be automatically merged.

@cmsbuild
cmsbuild merged commit 475de19 into cms-sw:master Aug 17, 2026
10 checks passed
@AdrianoDee

Copy link
Copy Markdown
Contributor Author

Note: the inputs are there

[adiflori@lxplus983 adiflori]$ dasgoclient --limit 0 --query 'file dataset=/RelValTTbar_14TeV/CMSSW_20_0_0-150X_mcRun4_realistic_v1_D127_GSOnly_NewInputs-v1/GEN-SIM'
/store/relval/CMSSW_20_0_0/RelValTTbar_14TeV/GEN-SIM/150X_mcRun4_realistic_v1_D127_GSOnly_NewInputs-v1/2590000/aa922eb2-9f88-4cf9-8456-9c19e297ef24.root
/store/relval/CMSSW_20_0_0/RelValTTbar_14TeV/GEN-SIM/150X_mcRun4_realistic_v1_D127_GSOnly_NewInputs-v1/2590000/9681c0cc-8467-4635-8d85-5af387d41e73.root

They would need to be staged by the bot to CERN.

@AdrianoDee
AdrianoDee deleted the new_d127_inputs_fix branch August 17, 2026 07:19
@smuzaffar

Copy link
Copy Markdown
Contributor

@AdrianoDee , bot will cache the files (which are accessed by IB workflows) if they are known to DAS and accessible via xrootd. As PR is merged so hopefully 11h00 IB will trigger the caching of input files opened by workflows

@AdrianoDee

AdrianoDee commented Aug 17, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for the explication @smuzaffar I was not sure about the mechanism.

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.

4 participants