-
Notifications
You must be signed in to change notification settings - Fork 696
feat: use prefix variable for POWERTOOLS_SERVICE_NAME in Lambda functions #4948
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
base: main
Are you sure you want to change the base?
feat: use prefix variable for POWERTOOLS_SERVICE_NAME in Lambda functions #4948
Conversation
835b1a0 to
d82d6f7
Compare
npalm
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@alexalbu001 thx, can you also fix the following places in code as well.
- modules/runners/pool/main.tf
- modules/runners/ssm-housekeeper.tf
PR looks good.
ffa0ba0 to
b02518c
Compare
|
@npalm should be good now |
|
Sorry for the late catch, but why do you think the service name should include the prefix as well. The initial idea was to set only the name of the service, since it is in context of the lambda the prefix is part of that. |
|
I understand the design philosophy of keeping the service name as a logical identifier separate from the deployment context, however, I see a few benefits to including the prefix:
|
Updates
POWERTOOLS_SERVICE_NAMEto use theprefixvariable instead of hardcoded values in pool, ami-housekeeper, scale-down, scale-up, webhook and dispatcher Lambda functions.Why
POWERTOOLS_SERVICE_NAMEshould too