-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Deprecate gp timeout extend
and review timeout defaults
#16083
Comments
Do we know how often Alternatively, even if we do decide to deprecate, we should at least make |
Not much. Usage is ~85 times a week.
💯 That sounds like a good first step. 🙋♂️ |
gp timeout extend
gp timeout extend
and review timeout defaults
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
It is already implemented now -- feature docs |
With the introduction of
gp timeout set
, and usage based pricing, thegp timeout extend
behaves in unexpected and unpredictable ways. When you set a new timeout to, let's say 15 hours, runninggp timeout extend
now arbitrarily reverts the timeout back to 3 hours. Soft deprecategp timeout extend
(e.g. remove from documentation and CLI to protect backwards compatibility) and instruct users to usegp timeout set 180m
as an alternative.The text was updated successfully, but these errors were encountered: