-
Notifications
You must be signed in to change notification settings - Fork 227
WIP: feat: add sanic support for elastic-apm python agent #522
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
WIP: feat: add sanic support for elastic-apm python agent #522
Conversation
Any updates on this PR? :D |
As referenced in #521, there were some issues with blocking code potentially causing issues in sanic. As far as I can tell, most of those have been fixed. Source code collection is now in a background thread as of #607. I think we can probably move forward on this, if @harshanarayana is willing and able. We added support for a couple of async frameworks recently as well, for reference: tornado, starlette, and aiohttp |
@dadoeyad Time for me to resurrect this PR and get this one sorted. I will take some time and get this one done over the weekend. @basepi I will take a look at the way the support has been added to the other frameworks and see how much of it can be leveraged into sanic. I will push the changes to this PR shortly |
Signed-off-by: Harsha Narayana <harsha2k4@gmail.com>
2a6a461
to
5ee5ed4
Compare
❕ Build Aborted
Expand to view the summary
Build stats
Trends 🧪Steps errors
Expand to view the steps failures
|
Is there any development on this? Any plans on adding native support for Sanic. This framework also is getting some serious attention |
Is there any development on this? |
awesome! |
Description
This PR is a quick prototype of what it would look like when it comes to enabling the elastic APM client support for
sanic
.Closes #521
To Do