-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
expand 2017 wf list for IB #18576
expand 2017 wf list for IB #18576
Conversation
A new Pull Request was created by @slava77 (Slava Krutelyov) for master. It involves the following packages: Configuration/PyReleaseValidation @kkousour, @kpedro88, @fabozzi, @cmsbuild, @GurpreetSinghChahal, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please propose to remove roughly the same cpu from elsewhere.. |
OK, I'll take a look. lack of this slows down tests and signoff of reco PRs (requires manual edits of das queries to drop CERN T2 and then fight with failures in xrootd). Until a few days ago the samples were apparently still at CERN, so it didn't come up sooner. |
just to be clear - I'm all for adding more 2017
… On May 5, 2017, at 1:41 PM, Slava Krutelyov ***@***.***> wrote:
OK, I'll take a look.
lack of this slows down tests and signoff of reco PRs (requires manual edits of das queries to drop CERN T2 and then fight with failures in xrootd). Until a few days ago the samples were apparently still at CERN, so it didn't come up.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
@davidlange6 I did recently remove ~10 IB WFs for 2023 in #18524 (though this was mostly to make room for the ever-expanding menagerie of new geometry variations...) |
ah - but if its IB tests, it matters less - sorry I misread IB for PR somehow
… On May 5, 2017, at 3:53 PM, Kevin Pedro ***@***.***> wrote:
@davidlange6 I did recently remove ~10 IB WFs for 2023 in #18524 (though this was mostly to make room for the ever-expanding menagerie of new geometry variations...)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
On 5/5/17 6:56 AM, David Lange wrote:
ah - but if its IB tests, it matters less - sorry I misread IB for PR
somehow
Yes, it's in the PR title :)
…> On May 5, 2017, at 3:53 PM, Kevin Pedro ***@***.***> wrote:
>
> @davidlange6 I did recently remove ~10 IB WFs for 2023 in #18524 (though this was mostly to make room for the ever-expanding menagerie of new geometry variations...)
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub, or mute the thread.
>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#18576 (comment)>, or
mute the thread
<https://github.com/notifications/unsubscribe-auth/AEdcbjR2tVSXtzi-7fgfkECUq-TOUhDBks5r2yqRgaJpZM4NRThy>.
|
On 5/5/17 6:56 AM, David Lange wrote:
ah - but if its IB tests, it matters less - sorry I misread IB for PR
somehow
> On May 5, 2017, at 3:53 PM, Kevin Pedro ***@***.***> wrote:
>
> @davidlange6 I did recently remove ~10 IB WFs for 2023 in #18524 (though this was mostly to make room for the ever-expanding menagerie of new geometry variations...)
If cleanup is still needed, even after the scope of the list is clear,
I was going to propose to thin out some of the 2015 data and maybe even
some run-1.
…>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub, or mute the thread.
>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#18576 (comment)>, or
mute the thread
<https://github.com/notifications/unsubscribe-auth/AEdcbjR2tVSXtzi-7fgfkECUq-TOUhDBks5r2yqRgaJpZM4NRThy>.
|
On May 5, 2017, at 5:21 PM, Slava Krutelyov ***@***.***> wrote:
On 5/5/17 6:56 AM, David Lange wrote:
> ah - but if its IB tests, it matters less - sorry I misread IB for PR
> somehow
>> On May 5, 2017, at 3:53 PM, Kevin Pedro ***@***.***> wrote:
>>
>> @davidlange6 I did recently remove ~10 IB WFs for 2023 in #18524 (though this was mostly to make room for the ever-expanding menagerie of new geometry variations...)
If cleanup is still needed, even after the scope of the list is clear,
I was going to propose to thin out some of the 2015 data and maybe even
some run-1.
anything thought to be useful to test should stay on the list... indeed 2015 data is getting less and less in that category I guess.
…
>>
>> —
>> You are receiving this because you were mentioned.
>> Reply to this email directly, view it on GitHub, or mute the thread.
>>
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <#18576 (comment)>, or
> mute the thread
> <https://github.com/notifications/unsubscribe-auth/AEdcbjR2tVSXtzi-7fgfkECUq-TOUhDBks5r2yqRgaJpZM4NRThy>.
>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1 |
add 2017
(ele guns 10, 35, 1000; pho guns 10, 35; mu guns 1, 10, 100, 1000, QCD 3TeV, QCD Flat)
This will also help to keep the input files in cmsbuild eos space,
since these are actively used in local MC checkout of pull requests.