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

Version 5.0.4 has an error #351

Closed
vadym-serdiuk opened this issue Jan 28, 2021 · 16 comments
Closed

Version 5.0.4 has an error #351

vadym-serdiuk opened this issue Jan 28, 2021 · 16 comments

Comments

@vadym-serdiuk
Copy link

Hi!
The version 5.0.4 made our build failed.

Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/django/core/management/base.py", line 330, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python3.8/site-packages/django/core/management/base.py", line 371, in execute
    output = self.handle(*args, **options)
  File "/usr/local/lib/python3.8/site-packages/django/core/management/commands/compilemessages.py", line 73, in handle
    basedirs.extend(settings.LOCALE_PATHS)
  File "/usr/local/lib/python3.8/site-packages/django/conf/__init__.py", line 82, in __getattr__
    self._setup(name)
  File "/usr/local/lib/python3.8/site-packages/django/conf/__init__.py", line 69, in _setup
    self._wrapped = Settings(settings_module)
  File "/usr/local/lib/python3.8/site-packages/django/conf/__init__.py", line 170, in __init__
    mod = importlib.import_module(self.SETTINGS_MODULE)
  File "/usr/local/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 961, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 783, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/app/app/__init__.py", line 3, in <module>
    from .celery import app as celery_app
  File "/app/app/celery.py", line 4, in <module>
    from celery import Celery
  File "/usr/local/lib/python3.8/site-packages/celery/local.py", line 491, in __getattr__
    module = __import__(self._object_origins[name], None, None,
  File "/usr/local/lib/python3.8/site-packages/celery/app/__init__.py", line 2, in <module>
    from celery import _state
  File "/usr/local/lib/python3.8/site-packages/celery/_state.py", line 15, in <module>
    from celery.utils.threads import LocalStack
  File "/usr/local/lib/python3.8/site-packages/celery/utils/__init__.py", line 6, in <module>
    from kombu.utils.objects import cached_property
  File "/usr/local/lib/python3.8/site-packages/kombu/utils/__init__.py", line 4, in <module>
    from .compat import fileno, maybe_fileno, nested, register_after_fork
  File "/usr/local/lib/python3.8/site-packages/kombu/utils/compat.py", line 16, in <module>
    from kombu.exceptions import reraise
  File "/usr/local/lib/python3.8/site-packages/kombu/exceptions.py", line 5, in <module>
    from amqp import ChannelError, ConnectionError, ResourceError
  File "/usr/local/lib/python3.8/site-packages/amqp/__init__.py", line 42, in <module>
    from .types import ChannelT, ConnectionT, MessageT
  File "/usr/local/lib/python3.8/site-packages/amqp/types.py", line 13, in <module>
    from .spec import method_sig_t
ImportError: cannot import name 'method_sig_t' from 'amqp.spec' (/usr/local/lib/python3.8/site-packages/amqp/spec.py)

Python 3.8
Django 3.1.5
Celery 5.0.5
Docker image: python:3.8

@kentuck13
Copy link

Same error.

@mintyPT
Copy link

mintyPT commented Jan 28, 2021

Same

@voegtlel
Copy link

voegtlel commented Jan 28, 2021

I wonder, aren't there any CI tests which would detect such kind of error? (I mean it already happens during importing the module)

It seems the codecov last ran 2 years ago, maybe that should be fixed as well?

@auvipy
Copy link
Member

auvipy commented Jan 28, 2021

strange! sorry guys might messed with some commit in the release time

@auvipy
Copy link
Member

auvipy commented Jan 28, 2021

fixing and releasing withing an hour

@auvipy
Copy link
Member

auvipy commented Jan 28, 2021

Hi guys please check this https://pypi.org/project/amqp/5.0.5/

@auvipy
Copy link
Member

auvipy commented Jan 28, 2021

really sorry guys for this. I m a terrible maintainer :(

@lithammer
Copy link

lithammer commented Jan 28, 2021

really sorry guys for this. I m a terrible maintainer :(

Shit happens 🤷‍♂️ Everyone make mistakes. At least you were quick to resolve the issue which makes you a good maintainer in my book.

@raphaelm
Copy link

raphaelm commented Jan 28, 2021

really sorry guys for this. I m a terrible maintainer :(

You're not! Without you, we'd need to do all this ourselves. Thank you for your work.

❤️

@jor-rit
Copy link
Contributor

jor-rit commented Jan 28, 2021

yeah, definitely not terrible :)
And #352 should get the CI working again, might help prevent this in the future.

openstack-mirroring pushed a commit to openstack/requirements that referenced this issue Jan 29, 2021
There is a reported error in amqp==5.0.4:
celery/py-amqp#351

Version 5.0.5 should fix it.

Closes-Bug: #1913572
Change-Id: I89efe56754a64988d2c61acb2f537fbfa9de178d
openstack-mirroring pushed a commit to openstack/openstack that referenced this issue Jan 29, 2021
* Update requirements from branch 'master'
  to 552d9808de499f373fe90c54604c69703d560b1f
  - Merge "Block amqp==5.0.4 and bump upper version to 5.0.5"
  - Block amqp==5.0.4 and bump upper version to 5.0.5
    
    There is a reported error in amqp==5.0.4:
    celery/py-amqp#351
    
    Version 5.0.5 should fix it.
    
    Closes-Bug: #1913572
    Change-Id: I89efe56754a64988d2c61acb2f537fbfa9de178d
@flying-sheep
Copy link
Contributor

You could yank the release, then nothing will accidentally install it.

@thedrow
Copy link
Member

thedrow commented Feb 1, 2021

@auvipy Please yank the release ASAP. @flying-sheep is right :)

@auvipy
Copy link
Member

auvipy commented Feb 1, 2021

@auvipy Please yank the release ASAP. @flying-sheep is right :)

ok, but never did it before, any pointer to guide to do that?

@thedrow
Copy link
Member

thedrow commented Feb 1, 2021

Just go to PyPi, check the list of versions and there will be a button next to the release to yank it.

@auvipy
Copy link
Member

auvipy commented Feb 3, 2021

Just go to PyPi, check the list of versions and there will be a button next to the release to yank it.

thanks, checked, but I only see that button on the packages to which I have admin access.

@thedrow
Copy link
Member

thedrow commented Feb 3, 2021

Yanked.

@thedrow thedrow closed this as completed Feb 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

10 participants