-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Update BeamSpotOnline tags for CRUZET [11_3_X] #34333
Update BeamSpotOnline tags for CRUZET [11_3_X] #34333
Conversation
A new Pull Request was created by @francescobrivio for CMSSW_11_3_X. It involves the following packages: Configuration/AlCa @malbouis, @andrius-k, @yuanchao, @kmaeshima, @ErnestaP, @ahmad3213, @rvenditti, @cmsbuild, @jfernan2, @tlampen, @pohsun, @francescobrivio, @tvami can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
test parameters:
|
@cmsbuild please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-e8cfd6/16470/summary.html Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
+1 |
thanks @jfernan2! A forward port is planned as soon as we also get confirmation of the candidates from HLT (hopefully tomorrow morning) and we can create versioned GTs to be put in autoCond. |
I am on it now. |
Pull request #34333 was updated. @malbouis, @andrius-k, @yuanchao, @kmaeshima, @ErnestaP, @ahmad3213, @rvenditti, @cmsbuild, @jfernan2, @tlampen, @pohsun, @francescobrivio, @tvami can you please check and sign again. |
test parameters:
|
@cmsbuild please test |
urgent |
@francescobrivio should we test again at P5 the new GTs 113X_dataRun3_HLT_v3/4 or are they copies of the candidates 113X_dataRun3_Express_Candidate_2021_07_02_* ? |
@jfernan2 They are exact copies of the candidates you already tested, see: |
OK, thanks for the confirmation!! |
forward port to #34348 |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-e8cfd6/16486/summary.html Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
+alca |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_11_3_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_12_0_X is complete. This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
This PR updates the BeamSpotOnline tags to the "production" version (i.e. removed the word
Test
from the tag name)to be used at HLT and in Express for CRUZET onwards:
onlinebeammonitor
DQM client (avaid crash from multiple sources)autoCond.py
HLT and Express candidate GTs which contain the new tags:As agreed with TRK DPG and POG (FYI @mtosi @mmusich) for now the tags only contain one payload with
since = 1
and with a "fakeBS":The GT diffs are as follow:
HLT: https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/113X_dataRun3_HLT_Candidate_2021_07_02_14_05_25/113X_dataRun3_HLT_v2
Express: https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/113X_dataRun3_Express_Candidate_2021_07_02_14_38_40/113X_dataRun3_Express_v3
IMPORTANT NOTE
This PR is needed to DQM in order to test the candidates, once both DQM and HLT confirm the candidates are fine they will be turned into versioned GTs and I will update this PR. This is needed for CRUZET.
FYI @qliphy @silviodonato @amassiro @boudoul
-EDIT-
DQM and HLT validated the candidates (HN message), so I made the versioned GTs for CRUZET by copying the candidates:
HLT: 113X_dataRun3_HLT_v3
Express: 113X_dataRun3_Express_v4
The new diffs are:
Run 3 data HLT
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/113X_dataRun3_HLT_v2/113X_dataRun3_HLT_v3
Run 3 data (express)
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/113X_dataRun3_Express_v3/113X_dataRun3_Express_v4
PR validation:
Tested with:
Back/Forward-port:
This will be forward-ported to master.