-
Notifications
You must be signed in to change notification settings - Fork 41
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
Does not work on circus==0.11.1, circus-web==0.5, tornado==4.2 because of wrong param for default value on get_arguments() #59
Comments
This is occurring due to this change in tornado: Along with a specific version not specified in requirements.txt for this package. In the main circus package, tornado is specified as For context, the error happens after you load the initial circus-web screen, then hit connect. It yields this exception:
Since I'm not familiar with this codebase nor Tornado, I just force installed tornado==3.2 (the current version at the time of this last release in 2014) and called it a day rather than trying to fix myself.
|
Just ran into this one as well when trying circus-web for the first time. |
It is strange since I released circus-web few days ago. Can you tell me which version of circus and circusweb you are using? |
|
Same here. Fresh install in an empty virtualenv, circus==0.12.0, circus-web==1.0.0. It's obviously triggered by tornadoweb/tornado@917c9ca and the workaround is to install tornado==3.2 as noted above. |
I am using tornado==3.2 but still facing same error as above:
|
Confirmed the issue still exists. Pinning to |
bump |
Issue still exists. Installing tornado==3.2 fixed it. |
circus 0.13.1, circus-web 1.0.0 same issue |
I'm using circus, version 0.14.0, circus-web, version 1.0.0 and chaussette, version 1.3.0 . |
Just bumped into this exact same issue...
FYI: e824d1a is still a cure. Apparently this fix somehow couldn't find its way to |
bumped into this exact same issue: |
If one of you wants to file a pull-request I will be happy to merge it and make a new release. |
@Natim I believe @narunask is exactly correct. Just making a pip release should work. |
It's definitely valid today - I installed both circus and circus-web for the first time today and ran into this problem. |
@waynew they haven't made a new release of circus-web. |
now is 2018-08-21, this issue still exist. |
2018-09-01, same issue still exist.
|
pinning to tornado==3.2.2 is solving this. but the issue still exists. |
circus==0.12.0, circus-web==1.0.0, tornado==3.2.2 have already solved my problems. Thanks. |
No description provided.
The text was updated successfully, but these errors were encountered: