-
Notifications
You must be signed in to change notification settings - Fork 68
Replay for 12_4_11_patch1 #4781
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
|
Can one of the admins verify this patch? |
|
The replay contains three runs to test all the fixes:
@germanfgv @jhonatanamado Will the replay take too long or could it be fine? |
|
@ebrondol this is way too big. 361971 and 361468 are 0.5PB of data. We can have a replay for 359762, which is a good pp run with all components in, as a general test. We can also have reduced replays with the bigger runs, in which we only process the affected stream. |
Looks like a good proposal to me! |
|
Is it possible to include the specific events that failed for these patches? I imagine not, so I guess we should just mention here any private tests that checked them. |
|
test syntax please |
|
@germanfgv Can we start the replay(s) you mentioned above? |
|
Yes, here you can follow the general replay with 359762 |
|
hi @germanfgv , if I see the grafana plots correctly, the replay with 359762 is going well. |
|
@ebrondol about the other replays: I also tried to start the replay with 361468, but discovered that we no longer have the streamers. Checking the deletion logs, it looks like we added the run to the exclusion list a bit too late. This means we cannot test this with replay, but we'll have relay on the tests done with the job's Pset. |
|
After the successful test, this was merged into production here #4784 |
Replay Request
Requestor
Erica Brondolin (as ORM)
Describe the configuration
Purpose of the test
Test the new CMSSW_12_4_11_patch1 release in view of the change in the T0 configuration.
T0 Operations cmsTalk thread
CMS Talk