-
Notifications
You must be signed in to change notification settings - Fork 54
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
disco: temporarily disable disco #2493
Conversation
oliver-sanders
commented
Nov 8, 2021
- Tornado 6+ does not support WSGI.
- Disable it for now, fix later.
- See rosie disco: wsgi - replace tornado with a wsgi compatible server #2492
* Tornado 6+ does not support WSGI. * Disable it for now, fix later. * See metomi#2492
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this have a changelog entry?
Eh, could do, it's only us who care about this feature. |
Put one in for the hell of it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've checked out the branch and can confirm this is disabled. I'm happy for merge once @MetRonnie is.
25471ab
to
e7c62e5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Hmm https://github.com/metomi/rose/runs/4164101189?check_suite_focus=true#step:5:161
Is that because for the docs build test we're not using the GitHub dev version of cylc-flow, instead we're using PyPI version? |
Might as well squeeze a fix for that in here too... |
Docs passing. |