-
Notifications
You must be signed in to change notification settings - Fork 68
Replay test with 12_1_0_pre3 #4607
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
Conversation
|
!!! Couldn't read commit file !!! |
|
@germanfgv I noticed "Syntax Check" and "Unit Tests" above show failure message but not obvious details. Can I just ignore them? |
|
@qliphy we are still updating our test to run with python3. They still don't work perfectly with our new agent. So yes, they can be ignored. I'll manually the replay and let you know about the results. |
| # Defaults for CMSSW version | ||
| defaultCMSSWVersion = { | ||
| 'default': "CMSSW_11_3_2" | ||
| 'default': "CMSSW_12_1_0_pre3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@qliphy shouldn't you also add the cmssw override for repacking (line 145) and express (line 159)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@francescobrivio Thanks. Now added.
Streamer files for run 344518 are not available. Using 343498 instead.
|
!!! Couldn't read commit file !!! |
|
!!! Couldn't read commit file !!! |
|
@qliphy @francescobrivio after this changes were made, we restarted the replay with the proper config. Around 50% of the jobs have finished with no issues so far. |
|
As reported in HyperNewas, the replay was successfull with the current configuration. I'll merge this as the current default replay config for Tier0. |
This reverts commit 098669a.
See #4605
Runs: 341169,341754,342154,344518
Release: CMSSW_12_1_0_pre3
GT: