Skip to content

feat: Upgrade to CDK 2.215.0. RustFunction now supports specifying a log group.#83

Merged
calavera merged 2 commits into
cargo-lambda:mainfrom
kjones:rust-function-log-group
Sep 18, 2025
Merged

feat: Upgrade to CDK 2.215.0. RustFunction now supports specifying a log group.#83
calavera merged 2 commits into
cargo-lambda:mainfrom
kjones:rust-function-log-group

Conversation

@kjones

@kjones kjones commented Sep 17, 2025

Copy link
Copy Markdown
Contributor

Fixes #82

Really only took upgrading the CDK to 2.215.0 and then the logGroup property was available for RustFunction.

calavera
calavera previously approved these changes Sep 17, 2025

@calavera calavera left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Thanks a lot for your help

@kjones

kjones commented Sep 17, 2025

Copy link
Copy Markdown
Contributor Author

Think I've got the yarn lockfile fixed now. All the commands in the .github/build.yml file now run locally for me.

@kjones

kjones commented Sep 18, 2025

Copy link
Copy Markdown
Contributor Author

@calavera Is there anything you need from me to get this released? I've been testing cdk synth/deploy to a dev environment with my local cargo-lambda-cdk build which works as expected.

@calavera calavera merged commit 2840913 into cargo-lambda:main Sep 18, 2025
1 check passed
@calavera

Copy link
Copy Markdown
Contributor

Thanks for your contribution @kjones. The library should be released automatically as soon as the build passes in the main branch.

@kjones kjones deleted the rust-function-log-group branch September 18, 2025 18:19
@kjones

kjones commented Sep 22, 2025

Copy link
Copy Markdown
Contributor Author

@calavera Just noticed that the release action didn't run to completion. Looks like an invalid NuGet API key. Deployment to pypi.org completed which is what I needed so I didn't notice the error.

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.

The Lambda Function logRetention property is deprecated. Allow specifying logGroup instead.

2 participants