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

feat(aws-lambda): Add Serverless Framework code snippet for Python #3830

Merged
merged 5 commits into from
Jul 19, 2021

Conversation

connorads
Copy link
Contributor

@connorads connorads commented Jul 3, 2021

Updating docs for two reasons:

  1. Want to double check that this is the recommended/correct way to do it 👀
  2. Hopefully this example helps other people using Python & serverless

Context

Just tried to use serverless with Python and the AWS Lambda Layer Integration.

Got something working, with a bit of trial and error, by combining the docs from:

Preview

https://sentry-docs-git-fork-connorads-patch-1.sentry.dev/product/integrations/aws-lambda/#serverless-framework

image

@vercel
Copy link

vercel bot commented Jul 3, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/sentry/sentry-docs/F51wC88oRWq36YHGafzduuYpxxHP
✅ Preview: https://sentry-docs-git-fork-connorads-patch-1.sentry.dev

@imatwawana imatwawana requested review from a team, ahmedetefy and vladanpaunovic and removed request for a team July 6, 2021 14:34
@HazAT
Copy link
Member

HazAT commented Jul 12, 2021

This LGTM but maybe @PeloWriter or @imatwawana could help to make this an include as we have everywhere else.

@ahmedetefy ahmedetefy changed the title AWS Lambda Layer Integration - Add Serverless Framework example for Python feat(aws-lambda): Add Serverless Framework code snippet for Python Jul 12, 2021
@connorads
Copy link
Contributor Author

Just resolved the merge conflicts 💃

Thanks @ahmedetefy for the changes and approval.
What else do we need to do to get this merged?

https://sentry-docs-git-fork-connorads-patch-1.sentry.dev/product/integrations/aws-lambda/#serverless-framework

@PeloWriter
Copy link
Contributor

Nothing further, thank you, @connorads. I'm tracking the suggestion to create include statements in this issue

@PeloWriter PeloWriter merged commit 5d24bea into getsentry:master Jul 19, 2021
taylangocmen pushed a commit that referenced this pull request Jul 22, 2021
…3830)

* Add Serverless Framework example for Python

* Refactored aws lambda code snippets for better readability

* Add missing functions conf + rename layersResource back to custom

Co-authored-by: Ahmed Etefy <[email protected]>
@connorads
Copy link
Contributor Author

I've now opened up a PR for some examples: getsentry/examples#101

@github-actions github-actions bot locked and limited conversation to collaborators Aug 8, 2021
@stephanie-anderson stephanie-anderson removed the request for review from vladanpaunovic March 5, 2024 12:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants