search

twitter / zktraffic

ZooKeeper protocol analyzer and stats gathering daemon
Apache License 2.0
167 stars 55 forks source link

zktraffic utils not working with python modules import error #72

Open dvrybak opened 4 years ago

dvrybak commented 4 years ago

$ sudo /usr/local/bin/zk-dump --iface ens192 --include-pings --unpaired Traceback (most recent call last): File "/usr/local/bin/zk-dump", line 29, in from twitter.common import app File "/usr/local/lib/python3.6/site-packages/twitter/common/app/init.py", line 39, in from .application import Application File "/usr/local/lib/python3.6/site-packages/twitter/common/app/application.py", line 39, in from twitter.common.process import daemonize File "/usr/local/lib/python3.6/site-packages/twitter/common/process/init.py", line 7, in from .process_provider_ps import ProcessProvider_PS File "/usr/local/lib/python3.6/site-packages/twitter/common/process/process_provider_ps.py", line 2, in from process_handle_ps import ProcessHandlePs ModuleNotFoundError: No module named 'process_handle_ps'

dvrybak commented 4 years ago

$ pip3 list Package Version

ansicolors 1.1.8 bottle 0.11.6 dpkt 1.9.2 hexdump 3.3 pex 1.6.11 pip 19.2.3 psutil 5.6.1 scapy 2.4.2 setuptools 39.2.0 six 1.12.0 tabulate 0.8.5 twitter.common.app 0.3.11 twitter.common.collections 0.3.11 twitter.common.contextutil 0.3.11 twitter.common.decorators 0.3.11 twitter.common.dirutil 0.3.11 twitter.common.exceptions 0.3.11 twitter.common.http 0.3.11 twitter.common.lang 0.3.11 twitter.common.log 0.3.11 twitter.common.options 0.3.11 twitter.common.process 0.3.11 twitter.common.string 0.3.11 twitter.common.util 0.3.11 zktraffic 0.2.0

rgs1 commented 4 years ago

@dvrybak hmm, i think https://github.com/twitter/zktraffic/commit/2643c59fc9d007472f24922aace8d84e5e5c28f3 needs a follow-up. We probably should get rid of twitter common unless it gets proper py3 support.

rgs1 commented 4 years ago

cc: @Yasumoto

Yasumoto commented 4 years ago

Yeah, I think the time has come to remove the twitter.common dependency.