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

chore(merge-back): 2.171.1 #32327

Merged
merged 4 commits into from
Nov 28, 2024
Merged

chore(merge-back): 2.171.1 #32327

merged 4 commits into from
Nov 28, 2024

Commits on Nov 27, 2024

  1. fix(cli): lambda hotswap fails if lambda:GetFunctionConfiguration a…

    …ction is not allowed (#32301)
    
    Closes #32219
    
    ### Reason for this change
    
    
    
    In SDKv3, the standard `waitUntilFunctionUpdated` function invokes the `GetFunctionConfiguration` API, as opposed to SDKv2, which invoked `GetFunction`. This means that consumers of SDKv3 must allow the `lambda:GetFunctionConfiguration` action in their IAM role policy.
    
    ### Description of changes
    
    
    
    Use a different waiter function provided by the SDK, which invokes `GetFunction` instead of `GetFunctionConfiguration`, and thus restoring required IAM permissions to what they were in SDKv2.
    
    See https://github.com/aws/aws-sdk-js-v3/blob/main/clients/client-lambda/src/waiters/waitForFunctionUpdatedV2.ts#L10
    
    > As opposed to https://github.com/aws/aws-sdk-js-v3/blob/main/clients/client-lambda/src/waiters/waitForFunctionUpdated.ts#L13
    
    ### Description of how you validated changes
    
    Manul test. Assumed a role with the following policies:
    
    ![Screenshot 2024-11-27 at 9 34 25](https://github.com/user-attachments/assets/69415c37-6fe8-44d3-972c-1373ec55f46e)
    
    ```console
     ❯ cdk deploy --hotswap                                                                                                                                                                                                                                            [09:29:11]
    
    ✨  Synthesis time: 2.72s
    
    ⚠️ The --hotswap and --hotswap-fallback flags deliberately introduce CloudFormation drift to speed up deployments
    ⚠️ They should only be used for development - never use them for your production Stacks!
    
    AwsCdkPlaygroundStack: deploying... [1/1]
    
    ✨ hotswapping resources:
       ✨ Lambda Function 'AwsCdkPlaygroundStack-Function76856677-7Rl7hiwwO5LQ'
    ❌  AwsCdkPlaygroundStack failed: TimeoutError: Resource is not in the expected state due to waiter status: TIMEOUT. Waiter has timed out.
    ```
    
    Then, run the CLI from the PR.
    
    ```console
    ❯ /Users/epolon/dev/src/github.com/aws/aws-cdk/packages/aws-cdk/bin/cdk deploy --hotswap                                                                                                                                                                          [10:03:00]
    
    ✨  Synthesis time: 3.46s
    
    ⚠️ The --hotswap and --hotswap-fallback flags deliberately introduce CloudFormation drift to speed up deployments
    ⚠️ They should only be used for development - never use them for your production Stacks!
    
    AwsCdkPlaygroundStack: deploying... [1/1]
    
    ✨ hotswapping resources:
       ✨ Lambda Function 'AwsCdkPlaygroundStack-Function76856677-7Rl7hiwwO5LQ'
    ✨ Lambda Function 'AwsCdkPlaygroundStack-Function76856677-7Rl7hiwwO5LQ' hotswapped!
    
     ✅  AwsCdkPlaygroundStack
    
    ✨  Deployment time: 12.72s
    
    Stack ARN:
    arn:aws:cloudformation:us-east-1:01234567890:stack/AwsCdkPlaygroundStack/22f2b380-a7cd-11ef-badd-0e08a8e0b5b1
    
    ✨  Total time: 16.19s
    
    >>> elapsed time 23s                                                                                                                                                                                                                                                          
    ```
    
    
    
    ### Checklist
    - [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md)
    
    ----
    
    *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
    iliapolo authored and rix0rrr committed Nov 27, 2024
    Configuration menu
    Copy the full SHA
    a073e93 View commit details
    Browse the repository at this point in the history
  2. chore(release): 2.171.1

    rix0rrr committed Nov 27, 2024
    Configuration menu
    Copy the full SHA
    8beaa20 View commit details
    Browse the repository at this point in the history
  3. chore(release): 2.171.1 (#32311)

    See CHANGELOG
    
    ----
    
    *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
    mergify[bot] authored Nov 27, 2024
    Configuration menu
    Copy the full SHA
    a95560c View commit details
    Browse the repository at this point in the history

Commits on Nov 28, 2024

  1. Configuration menu
    Copy the full SHA
    1347482 View commit details
    Browse the repository at this point in the history