Skip to content
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

zktraffic utils not working with python modules import error #72

Open
dvrybak opened this issue Oct 7, 2019 · 4 comments
Open

zktraffic utils not working with python modules import error #72

dvrybak opened this issue Oct 7, 2019 · 4 comments

Comments

@dvrybak
Copy link

dvrybak commented Oct 7, 2019

$ 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
Copy link
Author

dvrybak commented Oct 7, 2019

$ 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
Copy link
Collaborator

rgs1 commented Oct 7, 2019

@dvrybak hmm, i think 2643c59 needs a follow-up. We probably should get rid of twitter common unless it gets proper py3 support.

@rgs1
Copy link
Collaborator

rgs1 commented Oct 7, 2019

cc: @Yasumoto

@Yasumoto
Copy link
Collaborator

Yasumoto commented Oct 7, 2019

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants