Skip to content

Conversation

@smuzaffar
Copy link
Contributor

Backport changes from DEVEL branch

  • Move to x86-64-v3 as default microarch
  • Updates build rules to build publich binaries ( e.g cmsRun, edm) for additional micro-archs
  • Update zlib 1.2.13 to use consistent version for all archs

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @smuzaffar for branch IB/CMSSW_15_0_X/master.

@aandvalenzuela, @cmsbuild, @iarspider, @smuzaffar can you please review it and eventually sign? Thanks.
@antoniovilela, @mandrenguyen, @rappoccio, @sextonkennedy you are the release manager for this.
cms-bot commands are listed here

@cmsbuild
Copy link
Contributor

cmsbuild commented Dec 10, 2024

cms-bot internal usage

@smuzaffar
Copy link
Contributor Author

smuzaffar commented Dec 10, 2024

test parameters:

@smuzaffar
Copy link
Contributor Author

please test

@cmsbuild
Copy link
Contributor

-1

Failed Tests: Build
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2b2830/43365/summary.html
COMMIT: 7d02c2d
CMSSW: CMSSW_15_0_X_2024-12-10-1100/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmsdist/9564/43365/install.sh to create a dev area with all the needed externals and cmssw changes.

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:

You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2b2830/43365/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2b2830/43365/git-merge-result

Build

I found compilation error when building:

>> Checking EDM Class Transients in libDataFormatsCLHEP.so
@@@@ ----> OK  EDM Class Version 
>> Checking EDM Class Transients in libDataFormatsCSCDigi.so
@@@@ ----> OK  EDM Class Version 
>> Checking EDM Class Transients in libDataFormatsCSCRecHit.so
error: class 'CTPPSDetId' has a different checksum for ClassVersion 3. Increment ClassVersion to 4 and assign it to checksum 1554945820
error: class 'TotemRPDetId' has a different checksum for ClassVersion 3. Increment ClassVersion to 4 and assign it to checksum 1915602617
error: class 'CTPPSDiamondDetId' has a different checksum for ClassVersion 3. Increment ClassVersion to 4 and assign it to checksum 3253161212
error: class 'CTPPSPixelDetId' has a different checksum for ClassVersion 3. Increment ClassVersion to 4 and assign it to checksum 1640295022
error: class 'TotemTimingDetId' has a different checksum for ClassVersion 3. Increment ClassVersion to 4 and assign it to checksum 3647294361
Suggestion: You can run 'scram build updateclassversion' to generate src/DataFormats/CTPPSDetId/src/classes_def.xml.generated with updated ClassVersion


@smuzaffar
Copy link
Contributor Author

@makortel , this change has been tested in DEVEL IB. the build errors are related to the enabling of edm checks and these are reproducible when we do a full build.

I am merging this for 15.0.x default IBs now

@smuzaffar smuzaffar merged commit 3e4044a into IB/CMSSW_15_0_X/master Dec 11, 2024
@makortel
Copy link
Contributor

the build errors are related to the enabling of edm checks and these are reproducible when we do a full build.

I've seen these errors too on many recent cmsdist PRs. But why are these errors not visible in the IB dashboard?

@smuzaffar
Copy link
Contributor Author

@makortel , see #9568.

@smuzaffar smuzaffar deleted the microarch-v3 branch December 18, 2024 14:35
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