Feature: provide a mechanism to inform the caller that a quota is exceeded#200
Draft
MattMcL4475 wants to merge 3 commits intoga4gh:developfrom
Draft
Feature: provide a mechanism to inform the caller that a quota is exceeded#200MattMcL4475 wants to merge 3 commits intoga4gh:developfrom
MattMcL4475 wants to merge 3 commits intoga4gh:developfrom
Conversation
Contributor
Author
|
@patmagee what are your thoughts on the best way to handle this? |
|
@MattMcL4475 i would be in favour of a fail fast model. I think waiting for quota for a given period of time is okay, but beyond a "reasonable limit" the tes task should fail. I propose a new state |
|
I think this state would fit well onto a WES workflow as well btw |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Currently, when Azure Batch has no quota available, a TES Task in TES on Azure will stay in the
INITIALIZINGstate indefinitely until quota becomes available. This could be minutes, hours, or even days. TES needs a way to inform the caller why this is the case, so that the caller can update the UI with this additional information, and the user or IT admin knows they need to submit an Azure Support Request to increase their quota. Otherwise, they don't have visibility into why the task is not progressing.Ideally we actually want the caller to parse the string
Pending available quota: low-priority vCPUs, to recognize that there is a quota issue, and that the specific quota islow-priority vCPUs, so I'm also open to the idea of adding a specificstringproperty to the TES Task such asquotaTypeExceeded, and set it to a value such aslow-priority vCPUsorNVSv3 Series