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

Bump worker version #109

Closed

Conversation

fridex
Copy link
Contributor

@fridex fridex commented Jul 28, 2017

No description provided.

@fridex
Copy link
Contributor Author

fridex commented Jul 28, 2017

Needed as worker now requires joblib.

@jpopelka
Copy link
Contributor

jpopelka commented Jul 28, 2017

Hmm, strange:

coreapi-jobs            |   File "/usr/lib/python3.4/site-packages/google/cloud/_helpers.py", line 30, in <module>
coreapi-jobs            |     from google.protobuf import duration_pb2
coreapi-jobs            | ImportError: No module named 'google.protobuf'

running tests again ...

@jpopelka
Copy link
Contributor

[test this]

@jpopelka
Copy link
Contributor

Still the same, @fridex please check it.

@fridex
Copy link
Contributor Author

fridex commented Jul 28, 2017

It was discussed here: fabric8-analytics/fabric8-analytics-worker#115 (comment)

However on my machine:

$ docker run -it centos:7
[root@f883efadbc16 /]# yum install -y epel-release && yum install -y python34-pip && pip3 install protobuf
[root@f883efadbc16 /]# python3
Python 3.4.5 (default, May 29 2017, 15:17:55) 
[GCC 4.8.5 20150623 (Red Hat 4.8.5-11)] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from google.protobuf import duration_pb2
>>>

Maybe the import errors were caused for other imports in that case? @harjinder-hari could you elaborate?

@jpopelka
Copy link
Contributor

I've been looking at the protobuf related problem and even with all my google-fu I haven't been able to find better work-around than installing protobuf from source (as we do in worker's Dockerfile).
I'm afraid we need to do the same hack also in jobs, hence #113

@fridex
Copy link
Contributor Author

fridex commented Aug 1, 2017

Closing with respect to #113

@fridex fridex closed this Aug 1, 2017
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

Successfully merging this pull request may close these issues.

None yet

2 participants