Skip to content

Performance for Serverless on Python #2389

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

Closed
wants to merge 20 commits into from

Conversation

ajjindal
Copy link
Contributor

  • Supports Tracing
  • Captures function payload with the send-default-pii

@vercel
Copy link

vercel bot commented Sep 29, 2020

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/4n6mometq/sentry.dev
✅ Preview: https://sentry-docs-git-feat-serverlesstransactionpython.sentry.dev

Copy link
Contributor

@PeloWriter PeloWriter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Check out Sentry's [AWS sample apps](https://github.com/getsentry/examples/tree/master/aws-lambda/python) for detailed examples.

<Alert level="info" title="Note"><markdown>

If you are using another web framework inside of AWS Lambda, the framework might catch those exceptions before we get to see them. Make sure to enable the framework specific integration as well, if one exists. See [_Integrations_](/platforms/python/#integrations) for more information.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion to clarify:

Suggested change
If you are using another web framework inside of AWS Lambda, the framework might catch those exceptions before we get to see them. Make sure to enable the framework specific integration as well, if one exists. See [_Integrations_](/platforms/python/#integrations) for more information.
If you are using another web framework inside of AWS Lambda, the framework might catch those exceptions before they are sent to Sentry. As a result, we recommend enabling the framework-specific integration as well, if one exists. See [_Integrations_](/platforms/python/#integrations) for more information.

## Behavior

With the AWS Lambda integration enabled, the Python SDK will:

- Automatically report all exceptions from your Lambda Functions
- Automatically report all exceptions and transactions from your Lambda Functions
- You can control transactions sample rate using `traces_sample_rate`. You can learn more about transactions in [Distributed Tracing](/performance-monitoring/distributed-tracing/#traces-transactions-and-spans)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note - the content for this setting is forthcoming in PR 2376.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok let's not merge this PR before that one then. It seems that even with the current version of #2376 this will 404, so we'll have to re-review.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I took in the latest and the link still looks valid. @PeloWriter do you want me to update anything specific?

Copy link
Contributor

@PeloWriter PeloWriter Sep 30, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This link may be more helpful. So perhaps edit to:

@vercel vercel bot temporarily deployed to Preview September 29, 2020 22:15 Inactive
## Behavior

With the AWS Lambda integration enabled, the Python SDK will:

- Automatically report all exceptions from your Lambda Functions
- Automatically report all exceptions and transactions from your Lambda Functions
- You can control transactions sample rate using `traces_sample_rate`. You can learn more about transactions in [Distributed Tracing](/performance-monitoring/distributed-tracing/#traces-transactions-and-spans)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok let's not merge this PR before that one then. It seems that even with the current version of #2376 this will 404, so we'll have to re-review.

dcramer and others added 11 commits September 30, 2020 13:20
Docs has many broken/dead fragment URLs because an anchored section of a page moved or was renamed.
)

* fix(urls): Use relative URLs instead of linking to docs.sentry.io

* Revert relative URL change to api docs link

* Fix grouping & fingerprinting link

* Fix broken url on log4net docs
Co-authored-by: Fiona <[email protected]>
Co-authored-by: Bruno Garcia <[email protected]>
Co-authored-by: Manoel Aranda Neto <[email protected]>
dcramer and others added 6 commits September 30, 2020 16:02
- Expose dropdown for Product navigation
- Move all product pages into /product/
- Re-order sidebar to push relevant (vs marketing/discovery) docs to the top
…ocs (#2376)

Also a bunch of reorganization, and reworking of some (but not all) of the JS performance docs.
@untitaker
Copy link
Member

@ajjindal the rebase is messed up, this PR now contains too many unrelated changes to be reviewable

@ajjindal ajjindal closed this Oct 1, 2020
@ajjindal
Copy link
Contributor Author

ajjindal commented Oct 1, 2020

Creating a different PR

@ajjindal ajjindal deleted the feat/serverlessTransactionPython branch October 1, 2020 18:12
@github-actions github-actions bot locked and limited conversation to collaborators Dec 16, 2020
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.