sentry-javascript icon indicating copy to clipboard operation
sentry-javascript copied to clipboard

Unable to set relay logging level when using lambda layer

Open Lightbirdv opened this issue 2 years ago • 7 comments

Is there an existing issue for this?

  • [X] I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
  • [X] I have reviewed the documentation https://docs.sentry.io/
  • [X] I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which package are you using?

@sentry/serverless

SDK Version

7.32.1

Framework Version

No response

Link to Sentry event

No response

SDK Setup

Integrating sentry as a Lambda layer using CDK: const sentryLayer = LayerVersion.fromLayerVersionArn( scope, ${functionName}-sentrylayer, arn:aws:lambda:eu-central-1:943013980633:layer:SentryNodeServerlessSDK:108 );

Steps to Reproduce

  1. Integrating the lambda layer

Expected Result

Logs that are specifically coming from sentry itself.

Actual Result

Also getting the info logs from the relay middleware: [relay::setup] INFO [relay_server::actors::outcome] INFO [relay_server::actors::processor] INFO

please enable a lambda environment variable to set the logging level for relay.

Lightbirdv avatar Jan 24 '23 16:01 Lightbirdv

Hi @Lightbirdv thanks for writing in!

I'm not an expert on our serverless SDK but could you provide a link to which docs you followed to set up the layer?

Also, @antonpirker @sl0thentr0py since you have more experience around Relay on our serverless SDKs, would you mind taking a look at this?

Lms24 avatar Jan 24 '23 16:01 Lms24

I wanted to do this before shipping but I was on vacation back then and people shipped it already. So now this is a feature request and we'll need to pick t up, will take on our board.

sl0thentr0py avatar Jan 24 '23 16:01 sl0thentr0py

Alright thanks for the quick response. Looking forward to the feature!

Lightbirdv avatar Jan 24 '23 16:01 Lightbirdv

@Lightbirdv Can I just ask to assess importance - I assume you can somehow filter out our logs so it's rather a small annoyance than a breakage? Or does this impact you in a significant other way?

HazAT avatar Jan 24 '23 17:01 HazAT

@HazAT We are using datadog to filter it currently but since our infrastructure is pretty reliant on lambdas the cloudwatch logs are really cluttered with info logs on startup. So it would be nice if you test something to quickly search the cloudwatch logs without the relay spam.

Lightbirdv avatar Jan 24 '23 17:01 Lightbirdv

FWIW, this happens on cold starts and on recycles - so it is pretty much an ongoing problem.

DM1145 avatar Mar 15 '23 23:03 DM1145

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

getsantry[bot] avatar May 16 '24 07:05 getsantry[bot]