Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deploy new site based os mismatch check hook #9157

Merged
merged 1 commit into from
Apr 24, 2024

Conversation

smuzaffar
Copy link
Contributor

This deploys a new site based scram hook to check for scram/host OS mismatch. See cms-sw/cms-common#11

@smuzaffar
Copy link
Contributor Author

please test

@cmsbuild
Copy link
Contributor

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

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

@cmsbuild
Copy link
Contributor

cmsbuild commented Apr 24, 2024

cms-bot internal usage

@smuzaffar
Copy link
Contributor Author

+externals

It works as expected. on lxplus9

#set Default site hook to newly deployed hooks
export SCRAM_SITE_HOOKS_DIR=/cvmfs/cms-ci.cern.ch/week0/PR_e81d7ed9/etc/scramrc
> scram -a slc7_amd64_gcc700 p CMSSW_10_6_41
WARNING: Developer's area is created for architecture slc7_amd64_gcc700 while your current OS is EL9.
WARNING: You are trying to use SCRAM architecture 'slc7' on host with operating system 'el9'.
         This is not supported and likely will not work and you might get build/runtime errors. Please either
         - use correct SCRAM_ARCH to match your host's operating system.
         - OR use 'cmssw-el7' script to start a singularity container (http://cms-sw.github.io/singularity.html)
         - OR use host which has 'slc7' installed e.g. lxplus8 for el8, lxplus9 for el9 or lxplus7 for slc7.

@smuzaffar smuzaffar merged commit 504835b into IB/CMSSW_14_1_X/master Apr 24, 2024
8 of 9 checks passed
@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_14_1_X/master IBs after it passes the integration tests. This pull request will now be reviewed by the release team before it's merged. @sextonkennedy, @rappoccio, @antoniovilela (and backports should be raised in the release meeting by the corresponding L2)

@cmsbuild
Copy link
Contributor

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5cdfc6/39056/summary.html
COMMIT: d902ae1
CMSSW: CMSSW_14_1_X_2024-04-24-1100/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmsdist/9157/39056/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-5cdfc6/39056/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5cdfc6/39056/git-merge-result

Comparison Summary

Summary:

  • You potentially added 20 lines to the logs
  • Reco comparison results: 50 differences found in the comparisons
  • DQMHistoTests: Total files compared: 48
  • DQMHistoTests: Total histograms compared: 3319852
  • DQMHistoTests: Total failures: 7
  • DQMHistoTests: Total nulls: 195
  • DQMHistoTests: Total successes: 3319630
  • DQMHistoTests: Total skipped: 20
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 3605.085 KiB( 47 files compared)
  • DQMHistoSizes: changed ( 1000.0,... ): 127.810 KiB JetMET/MET
  • DQMHistoSizes: changed ( 1000.0 ): 43.835 KiB JetMET/Jet
  • DQMHistoSizes: changed ( 10224.0,... ): 43.851 KiB JetMET/Jet
  • DQMHistoSizes: changed ( 13234.0,... ): 0.016 KiB JetMET/Jet
  • Checked 202 log files, 165 edm output root files, 48 DQM output files
  • TriggerResults: no differences found

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.

2 participants