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(sdk): updating the aws-sdk to version 3.577.0 #5611
Conversation
Hey @marciocadev - what's the status of this PR? Once it's ready for review please let me know and move it out of draft, thank you |
Hey @staycoolcall911 I'm having an issue with the |
@staycoolcall911, I need some help here. Everything is okay, but the signedUrl test isn't working because I can't mock the getSignedUrl method. I created a project to test by copying this method, and the test I'm using in my project works perfectly. I'm not able to get this to work, could someone help me with this? Here's the project with the test working |
Hey @marciocadev - of course, I'll find someone to help you out. |
@marciocadev - @hasanaburayyan will reach out soon. |
@marciocadev my bad, I for some reason thought #5429 included this change. But I see it only upgraded the sdk to 3.490.0 |
I tried a few approaches (like mocks and spies) within our repo and also got stuck. After seeing it worked in https://github.com/marciocadev/AWSSignedUrlTest, I decided to instead try removing code from our repo to see what was necessary to get it working. I'm not quite sure why it fixes it, but if I comment out this line in our vitest config, it seems to fix the test. cc @MarkMcCulloh wing/libs/wingsdk/vitest.config.mts Line 5 in 845360a
|
Co-authored-by: Chris Rybicki <crybicki98@gmail.com>
@marciocadev @hasanaburayyan I feel for now it's better if we upgrade to the latest AWS SDK version and try to address the test mocking issue another time. I've pushed a commit that skips the relevant tests - let's try and get this across the finish line |
Signed-off-by: monada-bot[bot] <monabot@monada.co>
Signed-off-by: monada-bot[bot] <monabot@monada.co>
Spec tests passed: https://github.com/winglang/wing/actions/runs/9133888270 |
Thanks for contributing, @marciocadev! This PR will now be added to the merge queue, or immediately merged if |
Congrats! 🚀 This was released in Wing 0.73.53. |
This PR updates the aws-sdk past version 3.502, which reduces the cold start runtime on Lambdas according to the article by David Behroozi: [Coldstarts with the AWS Javascript 3.502 SDK](https://speedrun.nobackspacecrew.com/blog/2024/02/01/coldstarts-with-the-aws-javascript-3502-sdk.html) I did some tests and in a simple code like the one below, I managed to reduce the cold start from an average of 340ms to an average of 280ms (it's not much but every millisecond counts). ```wing bring cloud; let counter = new cloud.Counter() as "my-counter"; new cloud.Function(inflight () => { counter.inc(); }) as "my-function"; ``` - [x] Title matches [Winglang's style guide](https://www.winglang.io/contributing/start-here/pull_requests#how-are-pull-request-titles-formatted) - [x] Description explains motivation and solution - [ ] Tests added (always) - [ ] Docs updated (only required for features) - [ ] Added `pr/e2e-full` label if this feature requires end-to-end testing *By submitting this pull request, I confirm that my contribution is made under the terms of the [Wing Cloud Contribution License](https://github.com/winglang/wing/blob/main/CONTRIBUTION_LICENSE.md)*.
This PR updates the aws-sdk past version 3.502, which reduces the cold start runtime on Lambdas according to the article by David Behroozi: Coldstarts with the AWS Javascript 3.502 SDK
I did some tests and in a simple code like the one below, I managed to reduce the cold start from an average of 340ms to an average of 280ms (it's not much but every millisecond counts).
Checklist
pr/e2e-full
label if this feature requires end-to-end testingBy submitting this pull request, I confirm that my contribution is made under the terms of the Wing Cloud Contribution License.