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

Changed waiter.config.delay from 5 seconds to 10 seconds #2761

Closed
wants to merge 1 commit into from

Conversation

Seblat5ch
Copy link

The documentation below mentions the following with regards to: aws cloudformation wait stack-create-complete [1]
[1] http://docs.aws.amazon.com/cli/latest/reference/cloudformation/wait/stack-create-complete.html

"Wait until stack status is CREATE_COMPLETE. It will poll every 30 seconds until a successful state has been reached.
This will exit with a return code of 255 after 120 failed checks."

This should be technically be 30 seconds not 5 seconds or 10 seconds because:
120 calls at 30 seconds intervals will equal 3600 seconds/1 hour until the "Max attempts exceeded" which to confirms with CloudFormations 1 hour* timeout.

I understand that it is a fine line between performance (speed) and consistency, but unfortunately some services like CloudFront take time to build.

The documentation below mentions the following with regards to: aws cloudformation wait stack-create-complete [1]
[1] http://docs.aws.amazon.com/cli/latest/reference/cloudformation/wait/stack-create-complete.html

"Wait until stack status is CREATE_COMPLETE. It will poll every 30 seconds until a successful state has been reached. 
This will exit with a return code of 255 after 120 failed checks."

This should be technically be 30 seconds not 5 seconds or 10 seconds because: 
120 calls at 30 seconds intervals will equal 3600 seconds/1 hour until the "Max attempts exceeded" which to confirms with CloudFormations 1 hour* timeout.

I understand that it is a fine line between performance (speed) and consistency, but unfortunately some services like CloudFront take time to build.
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.

3 participants