-
Notifications
You must be signed in to change notification settings - Fork 649
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
feat: add ami-updater lambda function with tests and config #4488
base: main
Are you sure you want to change the base?
Conversation
- create new terraform module for ami updater lambda function - add comprehensive documentation in readme - implement all required resources including lambda, iam, eventbridge, and ssm - add variables for flexible configuration - include outputs for module integration
- implement core ami updater logic with ec2 and ssm integration - add comprehensive unit tests for all components - configure eslint and typescript for code quality - include vitest for testing framework - add aws powertools for logging and metrics - implement dry run mode for safe testing - add error handling and logging throughout
- add s3 bucket, key, and version variables for lambda deployment - make lambda zip path configurable via variable - add support for additional lambda tags - update lambda resource to handle both local and s3 deployments
- modify dist script to create zip file in dist/ami-updater.zip instead of root directory - ensure zip file location matches terraform module's expected path
…ling - update zip file location in package.json to match other modules - add ami-updater module configuration in root main.tf with consistent variable mapping - add ami_updater_lambda variables in root variables.tf for zip, memory and timeout - enable optional ami-updater functionality through enable_ami_updater variable
Created an PR to get the AMI ID via SSM supported via the launch template. You feedback would be very welcome. I think once we have that better done in the core. An auto update like this PR would be a welcome additon. See #4517 |
Hey @npalm! I’ve added a single question about your preference for using the |
Summary
This PR introduces a new AMI Updater Lambda function with comprehensive configuration, testing, and infrastructure setup.
Changes
Additional Notes