zktraffic icon indicating copy to clipboard operation
zktraffic copied to clipboard

zktraffic utils not working with python modules import error

Open dvrybak opened this issue 6 years ago • 4 comments

$ 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 avatar Oct 07 '19 09:10 dvrybak

$ 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

dvrybak avatar Oct 07 '19 09:10 dvrybak

@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 avatar Oct 07 '19 17:10 rgs1

cc: @Yasumoto

rgs1 avatar Oct 07 '19 18:10 rgs1

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

Yasumoto avatar Oct 07 '19 21:10 Yasumoto