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
nixos/moinmoin: fix module by switching to eventlet worker #108045
Conversation
The service was failing with: gunicorn[2192104]: [2020-12-31 13:35:28 +0000] [2192104] [ERROR] Exception in worker process gunicorn[2192104]: Traceback (most recent call last): gunicorn[2192104]: File "/nix/store/jmc14qf1sfnlhw27xyyj862ghkmdkj5a-python2.7-gunicorn-19.10.0/lib/python2.7/site-packages/gunicorn/arbiter.py", line 586, in spawn_worker gunicorn[2192104]: worker.init_process() gunicorn[2192104]: File "/nix/store/jmc14qf1sfnlhw27xyyj862ghkmdkj5a-python2.7-gunicorn-19.10.0/lib/python2.7/site-packages/gunicorn/workers/ggevent.py", line 196, in init_process gunicorn[2192104]: self.patch() gunicorn[2192104]: File "/nix/store/jmc14qf1sfnlhw27xyyj862ghkmdkj5a-python2.7-gunicorn-19.10.0/lib/python2.7/site-packages/gunicorn/workers/ggevent.py", line 65, in patch gunicorn[2192104]: monkey.patch_all(subprocess=True) gunicorn[2192104]: File "/nix/store/fysf67w3i8iv1hfvp536nl8jbzqyk1s7-python-2.7.18-env/lib/python2.7/site-packages/gevent/monkey.py", line 1160, in patch_all gunicorn[2192104]: from gevent import events gunicorn[2192104]: File "/nix/store/fysf67w3i8iv1hfvp536nl8jbzqyk1s7-python-2.7.18-env/lib/python2.7/site-packages/gevent/events.py", line 67, in <module> gunicorn[2192104]: from zope.interface import Interface gunicorn[2192104]: ImportError: No module named zope.interface
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
this should probably update to python3 instead zope implements a native namespace so there's not conflict in python3, but this isn't a feature in python2 |
Unfortunately moinmoin probably won't be ported to python3:( http://moinmo.in/Python3 |
@ofborg test moinmoin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks for the fix! It seems that 20.09 is also effected by this. Could you backport this PR? |
Motivation for this change
The service (and consequently
nixosTests.moinmoin
) was failing with:The
gevent
package does depend onzope_interface
however. I wasn't able to figure out why exactly cannot the module be imported and decided to switch to differentgunicorn
backend instead.Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)