Skip to content

Commit

Permalink
fix(release): ensure we publish Lambda Layers to the intended AWS reg…
Browse files Browse the repository at this point in the history
…ion (elastic#4172)

Before this change an errant AWS_REGION envvar would override the
AWS_DEFAULT_REGION envvar. Explcitly using --region=... will win.

Fixes: elastic#4171
Refs: elastic#4109
  • Loading branch information
trentm authored and fpm-peter committed Aug 20, 2024
1 parent f3d70f9 commit d3f1be0
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions .ci/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ publish: validate-layer-name validate-aws-default-region
@aws lambda \
--output json \
publish-layer-version \
--region="$(AWS_DEFAULT_REGION)" \
--layer-name "$(ELASTIC_LAYER_NAME)" \
--description "AWS Lambda Extension Layer for the Elastic APM Node.js Agent" \
--license "Apache-2.0" \
Expand All @@ -69,6 +70,7 @@ grant-public-layer-access: validate-layer-name validate-aws-default-region
@aws lambda \
--output json \
add-layer-version-permission \
--region="$(AWS_DEFAULT_REGION)" \
--layer-name "$(ELASTIC_LAYER_NAME)" \
--action lambda:GetLayerVersion \
--principal '*' \
Expand Down

0 comments on commit d3f1be0

Please sign in to comment.