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

apply timefactor scaling to nightlyflat too #1898

Merged
merged 1 commit into from Nov 9, 2022
Merged

apply timefactor scaling to nightlyflat too #1898

merged 1 commit into from Nov 9, 2022

Conversation

sbailey
Copy link
Contributor

@sbailey sbailey commented Nov 4, 2022

nightlyflat jobs were getting a runtime limit of 5 minutes, which is borderline for KNL on bad I/O days. Most job types get an additional timefactor scaling (3 in the case of KNL) to acknowledge that some systems are faster than others. Previously nightlyflat and arc jobs were excluded from this. For arcs that was to avoid ridiculously long job timelimit requests, while just accepting occasional timeouts and resubmissions. It's unclear why nightlyflat was also previously excluded.

This PR re-enables timefactor scaling for nightlyflat jobs, which results in 15 minute requests on KNL, which is still short-enough and a comfortable overhead to the typical 5 minutes runtime. We may not use KNL for productions ever again, but a few test jobs are still using it for some comparisons and this would be handy to have.

@sbailey sbailey added this to In progress in Himalayas via automation Nov 4, 2022
@sbailey sbailey merged commit 4d9b920 into main Nov 9, 2022
Himalayas automation moved this from In progress to Done Nov 9, 2022
@sbailey sbailey deleted the minjobtime branch November 9, 2022 02:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

None yet

1 participant