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

[PR #1790/2c917689 backport][stable-5] cloudformation_stack_set - Add a waiter to ensure that running operations against existing stacksets complete #1806

Commits on May 5, 2023

  1. cloudformation_stack_set - Add a waiter to ensure that running operat…

    …ions against existing stacksets complete (#1790)
    
    cloudformation_stack_set - Add a waiter to ensure that running operations against existing stacksets complete
    
    SUMMARY
    Add a waiter to ensure that running operations against existing stacksets complete. Current code would fail in cases where new instances need to be added since the previous update_stack_set(module, stack_params, cfn) would still be running.
    Fixes #1608
    ISSUE TYPE
    
    Bugfix Pull Request
    
    COMPONENT NAME
    cloudformation_stack_set
    ADDITIONAL INFORMATION
    I initially thought that the module was not idempotent since new instances wouldn't be added to my existing stack sets. Upon closer examination, the issue had to do with the fact that we had prior calls being made before adding new instances to existing stack sets:
    raise error_class(parsed_response, operation_name)\nbotocore.errorfactory.OperationInProgressException:
        An error occurred (OperationInProgressException) when calling the UpdateStackInstances operation:
        Another Operation on StackSet arn:aws:cloudformation:us-east-1:XXXXXX:stackset/aws-config-stackset:2bcb419a-f263-48ca-9fe0-cdef11fb59de is in progress
    The error got triggered because of a missing waiter after this operation:
    changed |= update_stack_set(module, stack_params, cfn)
    This change add a waiter function after the update operation, which, in turn, ensure that the subsequent call to add stack instances to the stack set properly run.
    
    Reviewed-by: Mark Chappell
    (cherry picked from commit 2c91768)
    rmahroua authored and patchback[bot] committed May 5, 2023
    Configuration menu
    Copy the full SHA
    42348b0 View commit details
    Browse the repository at this point in the history