-
Notifications
You must be signed in to change notification settings - Fork 486
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
Implement support for function-timeout configuration #1852
base: master
Are you sure you want to change the base?
Conversation
Hey @96malhar and @philasmar, could you please review? Thanks |
This is an interesting change for the test tool and wonder if would disrupt user's debugging in other ways. For example I can see users having Is the currently struggle that the value is |
Thanks for the reply @normj! |
Hey @normj @96malhar @philasmar, Thanks |
Is there any estimate when this could be deployed? I'm also waiting for this feature. |
Issue #665
Description of changes:
The templates deploy a
function-timeout
field which should correlate to the number of seconds until the lambda function times out and exits. This is a requirement for functions that rely on theILambdaContext.RemainingTime
as aCancellationTokenSoucre
. The remaining time is always set toTimeSpan.Zero
currently making it harder to test, or requiring to break and manually set the value.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.