dashboard-plugin icon indicating copy to clipboard operation
dashboard-plugin copied to clipboard

Dashboard integration significantly increases Lambda duration

Open vbichkovsky opened this issue 5 years ago • 8 comments

Enabling Dashboard integration via setting org and app properties in serverless.yml while using webpack packaging and aws-sdk as a runtime dependency, results in significant Lambda runtime duration when an error occurs in the handler.

serverless.yml

service: issue
frameworkVersion: '2'

org: your-org-here
app: sls-issue

plugins:
  - serverless-webpack

provider:
  name: aws
  runtime: nodejs12.x
  stage: prod
  region: eu-west-1
  memorySize: 256
  timeout: 10

functions:
  hello:
    handler: index.hello
    events:
      - http:
          path: /
          method: get

I've prepared a sample project in a public repo: https://github.com/vbichkovsky/sls-issue In order to expose a problem, the app has to be deployed and an API Gateway endpoint has to be called.

Installed version

Framework Core: 2.8.0 (local)
Plugin: 4.1.1
SDK: 2.3.2
Components: 3.2.7

vbichkovsky avatar Oct 28 '20 04:10 vbichkovsky

@vbichkovsky Is issue also present when webpack plugin is not used?

medikoo avatar Oct 28 '20 13:10 medikoo

@medikoo yes, it's present, just to lesser degree. I think it may have something to do with the size of the payload sent to the serverless dashboard back-end - it's much bigger when everything is bundled together (in case with webpack). Looks like this data is sent synchronously, hence increasing the duration of lambda.

I just ran the version deployed without using webpack, here are the stats:

  • with sls dashboard enabled: duration from a cold start - around 250ms, subsequent calls - around 40ms each
  • without sls dashboard: cold start - around 20ms, subsequent calls - around 2 ms

I guess it can be easily overlooked in such conditions, but in case with a big webpack package (I included whole aws-sdk in my example) the payload in CloudWatch logs is huge and it takes more than 2 seconds (to prepare/log/send it?)

The payload I'm referring to looks like this in CloudWatch logs:

INFO	SERVERLESS_ENTERPRISE {"c":true,"b":"H4sIAAAAAAAAA7VYC1PjOBL....

vbichkovsky avatar Oct 28 '20 19:10 vbichkovsky

@vbichkovsky thanks for the details, we will check what could cause that, and get back to you

medikoo avatar Oct 29 '20 08:10 medikoo

@vbichkovsky Data isn't actually sent to the back-end via HTTP request. The dashboard integration works by ingesting logs asynchronously from CloudWatch. So the only operation performed synchronously is preparing that payload to write to the log file.

There are a few things we can do to help troubleshoot this. First, if the payload in cloudwatch is huge - please try disabling gzip compression and posting the entire logged result. You can do that by adding the following custom block in the serverless.yml:

custom:
  enterprise:
    compressLogs: false

That should result in the full json payload being written out. Please then share it here.

I suspect that part of this issue is stackman, a dependency used to capture stack traces from the node environment.

astuyve avatar Oct 29 '20 14:10 astuyve

There you go, here are uncompressed logs for one invocation, with webpack plugin enabled. cloudwatch-logs.csv.log

vbichkovsky avatar Oct 29 '20 22:10 vbichkovsky

Here is another one, without the webpack plugin. cloudwatch-logs.no-webpack.csv.log

vbichkovsky avatar Oct 29 '20 22:10 vbichkovsky

Thanks @vbichkovsky! As I'm sure you've noticed; this appears to be an interesting interaction between stackman and webpack, where the webpacked version is 1000x larger than the non-webpacked version: image.

Generating an 8MB log file is likely quite time consuming (and also expensive both in terms of CloudWatch costs at $.50/GB ingested and compute time of the lambda invocation itself.)

Just to confirm - if there is no error in the invocation, is lambda duration impacted at all?

astuyve avatar Oct 29 '20 23:10 astuyve

No, it's only impacted when there is an error.

I think it's stackman, indeed, as it tries to provide context about the error and assumes that runnable code conststs of multiple relatively short lines whereas with webpack (and other bundlers) there is just one huge line of code with all the modules bundled together.

vbichkovsky avatar Oct 30 '20 01:10 vbichkovsky