-
Notifications
You must be signed in to change notification settings - Fork 0
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
SLC Deployer: Add timeout option to CLI #52
Comments
How would interactive users know what the timeout value should be? |
@ahsimb this option makes actually sense, because the time to wait depends on a lot of factors. And, not all of them we have under control or can detect. Factors
|
I agree to both comments: this option makes sense in my eyes, but still the users won't know what the timeout value should be in advance. Bad situation. I personally would prefer having a way to adjust the timeout than having no way. |
In the scope of ticket #50 a new
ExtractValidator
has been implemented.The current ticket requests adding options to the CLI so that an interactive user can control the timeout duration.
Proposal:
@click.option('--extract-timeout-minutes', type=int, default=10)
Open question:
The text was updated successfully, but these errors were encountered: