Add CoderdIneligiblePrebuilds alert
#34
Merged
Conversation
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
Signed-off-by: Danny Kopping <[email protected]>
dannykopping
force-pushed
the
dk/prebuilds-alerts
branch
from
88aee51 to
413aabe
Compare
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.
Closes coder/internal#561
Ineligibility means a prebuild cannot be claimed.
This is the worst situation to be in, because resources have been provisioned but they cannot be used.
We cannot know apriori how many prebuilds operators want to provision, but we can at least raise a
notifyalert when at least 1 prebuild is ineligible for 10m. Most startup scripts should complete in that amount of time, and if this is too short the timeout can be configured.Operators can also add other thresholds with particular values to suit their operational purposes.