Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use old layers without relay for arm64 lambdas #47381

Closed
sl0thentr0py opened this issue Apr 14, 2023 · 3 comments
Closed

Use old layers without relay for arm64 lambdas #47381

sl0thentr0py opened this issue Apr 14, 2023 · 3 comments

Comments

@sl0thentr0py
Copy link
Member

Problem Statement

relay (which is used as our AWS extension) only ships for intel so our new layers don't work on arm64 lambdas

related: getsentry/relay#1305

Solution Brainstorm

just use the old layers meanwhile if the architecture is arm64, see how best to show this on the UI etc

@antonpirker
Copy link
Member

Is this something we need do to immediately?
And is the plan do attach different layers depending on the architecture of the Lambda or just remove relay again from the Lambdas? (we should remove it anyhow, I talked to Jan and he said Relay is made for running on environments where it has control, and the freezing of the Lambda environment can destroy batch processing stuff in Relay, so the Lambda environment is kind of the wrong habitat for relay)

@sl0thentr0py
Copy link
Member Author

closing because we decided on removing relay from our layers completely. We hoped to use relay as an AWS extension, but sadly did not get the performance improvements we desired to get from having relay on the side, so we're reverting both python and JS layers to the old SDK-only layer.

In some future we might revisit building a dedicated low-overhead AWS extension.

@github-actions github-actions bot locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants