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

openstack oslo.messaging load failed due to amqp can't find module 'vine' in Python3.4 #91

Closed
glongwave opened this issue Jun 1, 2016 · 2 comments

Comments

@glongwave
Copy link
Contributor

Traceback (most recent call last):
File "/home/jenkins/workspace/gate-smaug-python34/smaug/tests/base.py", line 92, in setUp
self.messaging_conf = messaging_conffixture.ConfFixture(CONF)
File "/home/jenkins/workspace/gate-smaug-python34/.tox/py34/lib/python3.4/site-packages/oslo_messaging/conffixture.py", line 50, in init
'oslo_messaging_rabbit')
File "/home/jenkins/workspace/gate-smaug-python34/.tox/py34/lib/python3.4/site-packages/oslo_messaging/conffixture.py", line 25, in _import_opts
import(module)
File "/home/jenkins/workspace/gate-smaug-python34/.tox/py34/lib/python3.4/site-packages/oslo_messaging/_drivers/impl_rabbit.py", line 30, in
import kombu.connection
File "/home/jenkins/workspace/gate-smaug-python34/.tox/py34/lib/python3.4/site-packages/kombu/connection.py", line 19, in
from kombu import exceptions
File "/home/jenkins/workspace/gate-smaug-python34/.tox/py34/lib/python3.4/site-packages/kombu/exceptions.py", line 12, in
from amqp import ChannelError, ConnectionError, ResourceError
File "/home/jenkins/workspace/gate-smaug-python34/.tox/py34/lib/python3.4/site-packages/amqp/init.py", line 39, in
from .basic_message import Message # noqa
File "/home/jenkins/workspace/gate-smaug-python34/.tox/py34/lib/python3.4/site-packages/amqp/basic_message.py", line 29, in
from .serialization import GenericContent
File "/home/jenkins/workspace/gate-smaug-python34/.tox/py34/lib/python3.4/site-packages/amqp/serialization.py", line 34, in
from .exceptions import FrameSyntaxError
File "/home/jenkins/workspace/gate-smaug-python34/.tox/py34/lib/python3.4/site-packages/amqp/exceptions.py", line 21, in
from .five import python_2_unicode_compatible
File "/home/jenkins/workspace/gate-smaug-python34/.tox/py34/lib/python3.4/site-packages/amqp/five.py", line 14, in
import vine.five
ImportError: No module named 'vine'

@liusheng
Copy link

liusheng commented Jun 2, 2016

yes, I have also found this issue yesterday.

the reason of this issue I think it is wrong usage of "install_requires" parameter in setup.py#setup() method. the "install_requires" has been put a list of requirements from the requirements folder, the reqs() method in setup.py use a filter() method, but the "filter" method in python3 return a filter object(iterator) than a list(in pyhon2.7), it only can be consumed once. that cause the required "vine" package cannot be installed in python3 environment.

there is a similar issue:
pypa/pip#3162

so the "install_requires=reqs('default.txt')" can be changed to "install_requires=list(reqs('default.txt'))" or return a list in reqs() to fix this issue.

@glongwave
Copy link
Contributor Author

Thanks liu-sheng, that should be the root cause.

glongwave added a commit to glongwave/py-amqp that referenced this issue Jun 2, 2016
openstack-gerrit pushed a commit to openstack/openstack that referenced this issue Jun 3, 2016
Project: openstack/ceilometer  c28336d400e90c31d046970fd70fa46c82d7bd14

Fix the py34 jenkins job

The oslo.messaging<5.2.0(the v5.2.0 update change is in review[1])
depends on amqp>=1.4.0, in amqp-2.0 and amqp2.0.1, there is an issue[2]
that lead the amqp cannot be correctly installed in py34
environment(the depent package vine-1.1.0 of amqp won't bee installed),
that has borken the ceilometer py34 job[3]. we can wait the change[1]
merged and then bump the oslo.message version to avoid this issue. Because
Ceilometer don't participate the global-requirements, so we bump the
oslo.messaging to 5.2.0.

[1] https://review.openstack.org/#/c/324023/
[2] celery/py-amqp#91
[3] http://logs.openstack.org/41/323641/4/check/gate-ceilometer-python34/cf3f043/console.html

Change-Id: Icb34a7f0e03662868aa0ce6e060c5e951711b79e
openstack-gerrit pushed a commit to openstack/ceilometer that referenced this issue Jun 3, 2016
The oslo.messaging<5.2.0(the v5.2.0 update change is in review[1])
depends on amqp>=1.4.0, in amqp-2.0 and amqp2.0.1, there is an issue[2]
that lead the amqp cannot be correctly installed in py34
environment(the depent package vine-1.1.0 of amqp won't bee installed),
that has borken the ceilometer py34 job[3]. we can wait the change[1]
merged and then bump the oslo.message version to avoid this issue. Because
Ceilometer don't participate the global-requirements, so we bump the
oslo.messaging to 5.2.0.

[1] https://review.openstack.org/#/c/324023/
[2] celery/py-amqp#91
[3] http://logs.openstack.org/41/323641/4/check/gate-ceilometer-python34/cf3f043/console.html

Change-Id: Icb34a7f0e03662868aa0ce6e060c5e951711b79e
binux added a commit to binux/pyspider that referenced this issue Jun 5, 2016
@ask ask closed this as completed in 04dc894 Jun 7, 2016
tanaypf9 pushed a commit to tanaypf9/pf9-requirements that referenced this issue May 20, 2024
Patch Set 1:

the oslo.messaging include a change[1] which constraint the amqp version to <2.0. in amqp-2.0 and amqp-2.0.1, there is an issue[2] that will cause amqp cannot be correctly installed in python3.4 environment. this issue has been broken the py34 jenkins job[3]. so this should be merged ASAP and then bump the version of oslo.messaging to >=5.2.0 in requirements.txt

any thoughts ?


[1] https://review.openstack.org/#/c/321704/
[2] celery/py-amqp#91
[3] http://logs.openstack.org/41/323641/4/check/gate-ceilometer-python34/cf3f043/console.html

Patch-set: 1
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

2 participants