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

Add schedulingPriority option to aws batch (use with shareIdentifier) #3505

Merged
merged 3 commits into from
Dec 22, 2022

Conversation

bentsherman
Copy link
Member

Apparently, the new fair-share scheduling in AWS Batch requires both a shareIdentifier and a schedulingPriority (integer from 0 to 9999), otherwise you get an error:

Either schedulingPriority in job definition or schedulingPriorityOverride should be specified. (Service: AWSBatch; Status Code: 400; Error Code: ClientException; Request ID: 6e66d921-a989-41e2-b1d3-b9c8b2c792a7; Proxy: null)

This PR adds that option with a default of 0. Originated from a customer ticket.

docs/config.rst Outdated Show resolved Hide resolved
Signed-off-by: Paolo Di Tommaso <[email protected]>
@pditommaso pditommaso merged commit 119fe6b into master Dec 22, 2022
@pditommaso pditommaso deleted the ben/aws-batch-scheduling-priority branch December 22, 2022 15:46
pditommaso added a commit that referenced this pull request Dec 22, 2022
…er`) (#3505)

Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Co-authored-by: Paolo Di Tommaso <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
abhi18av pushed a commit to abhi18av/nextflow that referenced this pull request Jan 10, 2023
…er`) (nextflow-io#3505)

Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Co-authored-by: Paolo Di Tommaso <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
pditommaso added a commit that referenced this pull request Jan 14, 2023
…er`) (#3505)

Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Co-authored-by: Paolo Di Tommaso <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
l-modolo pushed a commit to l-modolo/nextflow that referenced this pull request Jan 25, 2023
…er`) (nextflow-io#3505)

Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Co-authored-by: Paolo Di Tommaso <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants