#3992 Circular import on logcom_1 and worker_1 containers
Closed: Fixed 5 years ago by lenkaseg. Opened 5 years ago by lenkaseg.

worker_1 | Traceback (most recent call last):
worker_1 | File "/usr/bin/celery", line 11, in <module>
worker_1 | load_entry_point('celery==4.0.2', 'console_scripts', 'celery')()
worker_1 | File "/usr/lib/python2.7/site-packages/celery/main.py", line 14, in main
worker_1 | _main()
worker_1 | File "/usr/lib/python2.7/site-packages/celery/bin/celery.py", line 326, in main
worker_1 | cmd.execute_from_commandline(argv)
worker_1 | File "/usr/lib/python2.7/site-packages/celery/bin/celery.py", line 488, in execute_from_commandline
worker_1 | super(CeleryCommand, self).execute_from_commandline(argv)))
worker_1 | File "/usr/lib/python2.7/site-packages/celery/bin/base.py", line 279, in execute_from_commandline
worker_1 | argv = self.setup_app_from_commandline(argv)
worker_1 | File "/usr/lib/python2.7/site-packages/celery/bin/base.py", line 481, in setup_app_from_commandline
worker_1 | self.app = self.find_app(app)
worker_1 | File "/usr/lib/python2.7/site-packages/celery/bin/base.py", line 503, in find_app
worker_1 | return find_app(app, symbol_by_name=self.symbol_by_name)
worker_1 | File "/usr/lib/python2.7/site-packages/celery/app/utils.py", line 358, in find_app
worker_1 | sym = imp(app)
worker_1 | File "/usr/lib/python2.7/site-packages/celery/utils/imports.py", line 101, in import_from_cwd
worker_1 | return imp(module, package=package)
worker_1 | File "/usr/lib64/python2.7/importlib/init.py", line 37, in import_module
worker_1 | import(name)
worker_1 | File "/code/pagure/lib/tasks.py", line 32, in <module>
worker_1 | import pagure.lib.git
worker_1 | File "/code/pagure/lib/git.py", line 38, in <module>
worker_1 | import pagure.lib.query
worker_1 | File "/code/pagure/lib/query.py", line 65, in <module>
worker_1 | from pagure.lib import tasks
worker_1 | ImportError: cannot import name tasks

=============================================================
Traceback (most recent call last):
logcom_1 | File "/usr/bin/celery", line 11, in <module>
logcom_1 | load_entry_point('celery==4.0.2', 'console_scripts', 'celery')()
logcom_1 | File "/usr/lib/python2.7/site-packages/celery/main.py", line 14, in main
logcom_1 | _main()
logcom_1 | File "/usr/lib/python2.7/site-packages/celery/bin/celery.py", line 326, in main
logcom_1 | cmd.execute_from_commandline(argv)
logcom_1 | File "/usr/lib/python2.7/site-packages/celery/bin/celery.py", line 488, in execute_from_commandline
logcom_1 | super(CeleryCommand, self).execute_from_commandline(argv)))
logcom_1 | File "/usr/lib/python2.7/site-packages/celery/bin/base.py", line 279, in execute_from_commandline
logcom_1 | argv = self.setup_app_from_commandline(argv)
logcom_1 | File "/usr/lib/python2.7/site-packages/celery/bin/base.py", line 481, in setup_app_from_commandline
logcom_1 | self.app = self.find_app(app)
logcom_1 | File "/usr/lib/python2.7/site-packages/celery/bin/base.py", line 503, in find_app
logcom_1 | return find_app(app, symbol_by_name=self.symbol_by_name)
logcom_1 | File "/usr/lib/python2.7/site-packages/celery/app/utils.py", line 358, in find_app
logcom_1 | sym = imp(app)
logcom_1 | File "/usr/lib/python2.7/site-packages/celery/utils/imports.py", line 101, in import_from_cwd
logcom_1 | return imp(module, package=package)
logcom_1 | File "/usr/lib64/python2.7/importlib/init.py", line 37, in import_module
logcom_1 | import(name)
logcom_1 | File "/code/pagure/lib/tasks_services.py", line 31, in <module>
logcom_1 | import pagure.lib.query
logcom_1 | File "/code/pagure/lib/query.py", line 66, in <module>
logcom_1 | from pagure.lib import tasks_services
logcom_1 | ImportError: cannot import name tasks_services


I'm trying to solve it.

the PYTHONPATH is not specified or does not include the path to /code/pagure?

it does include /code . The problem comes from pagure/lib/tasks.py and pagure/lib/tasks_services.py importing pagure/lib/query.py which in turn imports back pagure/lib/tasks.py and pagure/lib/tasks_services.py

Ah yes, I ran into this not long ago, I believe it's fixed in git now

#3958 and #3959 are the ones doing this iirc

I pull from master and the errors are still there:
https://pastebin.com/y5hTV99w

Is this still happening? I know we fixed some of these recently :)

No, it's not. I think it's fixed :)

Metadata Update from @lenkaseg:
- Issue close_status updated to: Fixed
- Issue status updated to: Closed (was: Open)

5 years ago

Login to comment on this ticket.

Metadata