8000 Comparing v2.171.0...v2.171.1 · aws/aws-cdk · GitHub
[go: up one dir, main page]

Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: aws/aws-cdk
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v2.171.0
Choose a base ref
...
head repository: aws/aws-cdk
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v2.171.1
Choose a head ref
  • 3 commits
  • 7 files changed
  • 3 contributors

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
Loading
0