gunicorn with gevent worker - is this supported?

[amir-hadi]

We see 'mixed' events in our Sentry installation. We recently activated the gevent worker for gunicorn, to get more concurrent requests (our service is highly IO dependent).

In the docs on sentry.io I couldn't find any documentation if gevent workers are supported or not.

I have seen some older tickets that state that it is not supported at all, but not sure if this is still the case? If it is not supported, how could we make it supported? I guess we would need to put tracking information into a Greenlet local? Would that be the right way to do it?

Thanks for any information.

[untitaker]

@amir-hadi do you have the gevent monkeypatches enabled (i.e. gevent monkeypatching the stdllib)? Those are necessary for sure.

[amir-hadi]

To my knowledge the Gevent worker of Gunicorn is patching everything for us, see this line in the Gunicorn source code. The call of self.patch() calls gevent.monkey.patch_all().

[untitaker]

Understood, in this case everything should be supported. At this point it becomes a bug. Which Python version is that?

[amir-hadi]

We are using Python 3.7.3 in our docker containers, but I am not sure if this is a bug on the UI or not. The code path we are seeing is impossible to reach for the API URL path visible in Sentry. Also, it seems like that impossible (for our system) combinations of Browser information is being done (okhttp client for native apps and Chrome for one of our management UIs).

Let me know how I can help you further to track this issue down.

[untitaker]

All of this is likely an artifact of leaking scopes. I will take a look soon.

…

On Fri, Jun 21, 2019, 09:47 Amir Hadi ***@***.***> wrote: We are using Python 3.7.3 in our docker containers, but I am not sure if this is a bug on the UI or not. The code path we are seeing is impossible to reach for the API URL path visible in Sentry. Also, it seems like that impossible (for our system) combinations of Browser information is being done (okhttp client for native apps and Chrome for one of our management UIs). Let me know how I can help you further to track this issue down. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#392?email_source=notifications&email_token=AAGMPRJYYCNOSZJJMNBEXJLP3SBPZA5CNFSM4HZKQAL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYHXABY#issuecomment-504328199>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAGMPRMRUE26HVMWKL2JR5TP3SBPZANCNFSM4HZKQALQ> .

[untitaker]

@amir-hadi I think you're running into gevent/gevent#1407, opened a PR

[amir-hadi]

@untitaker This basically means we cannot do anything about it right now I guess.

[untitaker]

@amir-hadi No, I will release a workaround in the SDK soon

[michaelschem]

Was there ever a workaround or patch released on this? Running into the same issue.

[antonpirker]

I am not sure, but I guess not.

Could you put together a minimal project where we can reproduce the problem so we can investigate this?

[untitaker]

@michaelschem those sort of issues are very sensitive to the combination of CPython version and gevent version. Especially in recent times there's been changes to CPython that break gevent, and then the latter's maintainers have to play catch-up. I would recommend creating a new issue or trying out different version combinations.