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

Python 3 support? #28

Open
scythargon opened this issue Dec 24, 2019 · 4 comments
Open

Python 3 support? #28

scythargon opened this issue Dec 24, 2019 · 4 comments
Assignees

Comments

@scythargon
Copy link

I thought you said that it is supported, but:

$ celerymon
Traceback (most recent call last):
  File "/home/argon/.Envs/burgundy/bin/celerymon", line 5, in <module>
    from celerymon.bin.celerymon import main
  File "/home/argon/.Envs/burgundy/lib/python3.6/site-packages/celerymon/bin/celerymon.py", line 94
    except Exception, exc:
                    ^
SyntaxError: invalid syntax
@auvipy
Copy link
Owner

auvipy commented Dec 24, 2019

it's still not ready for use!!

@scythargon
Copy link
Author

Got it, thank you:)
Celery lacks good monitoring so badly.

@auvipy
Copy link
Owner

auvipy commented Dec 24, 2019

yes! that's why I will take on this!

@mcepl
Copy link

mcepl commented Jan 10, 2022

Well, at least 2to3 claims that only change which is necessary is:

---
 celerymon/bin/celerymon.py |    5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

--- a/celerymon/bin/celerymon.py
+++ b/celerymon/bin/celerymon.py
@@ -91,9 +91,8 @@ class MonitorCommand(Command):

             try:
                 monitor.start()
-            except Exception, exc:
-                logger.error('celerymon raised exception %r',
-                             exc, exc_info=True)
+            except Exception:
+                logger.exception('celerymon raised exception')
             except KeyboardInterrupt:
                 pass

bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this issue Jan 10, 2022
… 11 via SR 945417

https://build.opensuse.org/request/show/945417
by user mcepl + dimstar_suse
- Update to version 1.0.3+git.1572430376.dac5d9f:
  * Unable to start as python manage.py celery mon - fixed.
  * fixing state imports
  * updating file permissions for celerymon.py
  * unrolling generators before json encoding
  * adding a global state object
  * adding unrolling for islice objects
  * fixing isinstance
  * Allow the page to be viewed on HTTPS as well
- Add fix_py2k_bug.patch removing py2k-based syntax error
  (bsc#1192677, gh#auvipy/celery-flower#28)
@auvipy auvipy self-assigned this Jan 11, 2022
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