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

Stop the user in case they add resources that exceed their limit #1355

Merged
merged 1 commit into from
Dec 6, 2023

Conversation

akalipetis
Copy link
Member

Since the current API does not provide a way to stop an update, this might lead in a state where the environment is not working and even doing forks will initialize new environment in this broken state.

This PR tries to calculate if the requested update will exceed the current user limit and if it does, it stops the operation. In case the user wants to proceed, they can use the --force argument.

@akalipetis akalipetis requested a review from pjcdawkins December 5, 2023 16:05
Since the current API does not provide a way to stop an update, this might lead in a state where the environment is not working and even doing forks will initialize new environment in this broken state.

This PR tries to calculate if the requested update will exceed the current user limit and if it does, it stops the operation. In case the user wants to proceed, they can use the `--force` argument.
@akalipetis akalipetis force-pushed the feature/block-usage-overage branch from f781f1f to cff9166 Compare December 5, 2023 20:50
@akalipetis akalipetis merged commit f2183fb into main Dec 6, 2023
2 checks passed
@akalipetis akalipetis deleted the feature/block-usage-overage branch December 6, 2023 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant