getsentry / sentry-javascript

Official Sentry SDKs for JavaScript
https://sentry.io
MIT License
7.99k stars 1.57k forks source link

Unable to set relay logging level when using lambda layer #6921

Closed Lightbirdv closed 5 months ago

Lightbirdv commented 1 year ago

Is there an existing issue for this?

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.

Lms24 commented 1 year ago

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?

sl0thentr0py commented 1 year ago

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.

Lightbirdv commented 1 year ago

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

HazAT commented 1 year ago

@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?

Lightbirdv commented 1 year ago

@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.

DM1145 commented 1 year ago

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

getsantry[bot] commented 6 months ago

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 🥀