openedx / xblock-lti-consumer

GNU Affero General Public License v3.0
27 stars 80 forks source link

feat: additional performance tracing #443

Closed Zacharis278 closed 7 months ago

Zacharis278 commented 7 months ago

We're spending over a second initializing the consumer object which slows down every endpoint. If that's our issue it seems like loading these keys is the only potential cause in that code path. Logging each step to confirm.

codecov[bot] commented 7 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (fdfbe5b) 97.85% compared to head (47e5aad) 97.85%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #443 +/- ## ======================================= Coverage 97.85% 97.85% ======================================= Files 77 77 Lines 6701 6705 +4 ======================================= + Hits 6557 6561 +4 Misses 144 144 ``` | [Flag](https://app.codecov.io/gh/openedx/xblock-lti-consumer/pull/443/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/openedx/xblock-lti-consumer/pull/443/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx) | `97.85% <100.00%> (+<0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.