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

Timeout Implementation without schema values and entry point values #245

Open
carpnick opened this issue Jan 11, 2023 · 0 comments
Open
Labels
enhancement New feature or request

Comments

@carpnick
Copy link

carpnick commented Jan 11, 2023

Looking at the contract as written, how do I guarantee the following lines?(see quoted below). From what I can tell, no plugin(this one included) contains the original start time of the resource call. Is it my handler's problem to track original call time in the callback context? Shouldnt this be included on the generated classes or within the framework Models? I can then just interrogate and respond with error, or even better let framework respond with error... Right now as written in contract - we dont know when to return since we do not have the start time and the resource schema of timeoutinMinutes is not exposed to us via any generated classes, correct? I see it is used to set maximum time on assuming the role only. That doesnt allow us to honor contract for less than 60 minutes for a single request: see here

As I see it we need at least 2 values to make the implementation somewhat generic:

  • Schema TimeoutInMinutes time needs to be exposed to internal AWS python plugin. Also you need to expose original request time as well. Both of these together will allow you to expose a real variable handlerEndTime that is then read/used by customer handlers to exit reinvoke logic with a timeout error. This is the lates time that a handler can run based on its own schema.
  • Handler current entry time - this gets reset for every handler native call - we can use it to make sure private handlers return in correct time frame of 60 seconds or 30 seconds based on the different Action types.

Contract Lines

A create handler MUST return FAILED progress event if it can't reach the desired-state within the timeout specified in the resource schema.

An update handler MUST return FAILED progress event if it can't reach the desired-state within the timeout specified in the resource schema.

A delete handler MUST return FAILED progress event if it can't reach the desired-state within the timeout specified in the resource schema.

Originally posted by @carpnick in #243

@ericzbeard ericzbeard added the enhancement New feature or request label Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants