aws-samples / aws-secure-environment-accelerator

The AWS Secure Environment Accelerator is a tool designed to help deploy and operate secure multi-account, multi-region AWS environments on an ongoing basis. The power of the solution is the configuration file which enables the completely automated deployment of customizable architectures within AWS without changing a single line of code.
Apache License 2.0
725 stars 233 forks source link

[BUG][Functional] Perimeter ALB Forwarding Lambda Function Error: Runtime.Unknown #1203

Closed Ray-PHSA closed 6 months ago

Ray-PHSA commented 9 months ago

Required Basic Info

Describe the bug The perimeter ALB function that constantly checks the internal ALB targets for IP address changes started failing after upgrading to 1.5.8-d. The issue is related to the Lambda runtime upgrade to Nodejs 18.

The function name is: ASEA-Perimeter-Phase1-Vpc-PerimeteralbIpForwarding-tBMt3ocTnJHS

Failure Info

Required files

Steps To Reproduce

  1. Make any change to the external ALB rules in the DynamoDB table
  2. See the errors in the Lambda logs.

Expected behavior The Lambda should work normally and it should update the ALB targets (with the correct IP addresses) whenever they change.

Screenshots If applicable, add screenshots to help explain your problem.

Additional context Add any other context about the problem here.

Ray-PHSA commented 9 months ago

Hi Brian, we fixed the Prod issue by increasing the Lambda memory to 512 MB. Please consider implementing that via code. Thanks!

ebellavance commented 8 months ago

Thank's Ray to report the problem. We just updated to 1.5.8-d and got the problem. We also increased the lambda memory to 512mb.

Do you know if running the statemachine (like creating a new account) will overide the change?

Regards,

Eric

Ray-PHSA commented 8 months ago

Hi, I'm not sure, I haven't run it since fixing the Lambda memory manually. I assume (based on the nature of CFN) that if the CFN synthesized template is not changing, CFN will not detect a drift and will leave the manual fix alone. Once you release 1.5.8-e (with the memory increase) then CFN will see a change and will attempt to update the function, then the function will be 'updated' from 512 (already done manually) to 512 (from the code) and the drift will be gone. BTW, how's the LZA upgrade script going? Thanks!

ebellavance commented 8 months ago

Ho, i'm not involved with ASEA or LZA development, i'm a customer using ASEA for his landing zone :)

johnathan-tracz commented 7 months ago

We are noticing that newly created forwarding rules do not seem to work after this update. We have increased the memory on the Lambda function to 512MB which got rid of the errors, but if we create a new forwarder, we see the following behaviour:

When checking the table entry in DynamoDB, the targetGroupIpAddresses entry never gets updated with any addresses.

Is anyone else seeing this behaviour?

archikierstead commented 6 months ago

Fixed in 1.5.9