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
[11.0][MIG] dead_mans_switch_client #1277
[11.0][MIG] dead_mans_switch_client #1277
Conversation
* [ADD] dead_mans_switch_client, dead_mans_switch_server * OCA Transbot updated translations from Transifex * Add demo system parameter and some fixes to README. Fix OCA#314. * Add timeout to urlopen in order to avoid "freeze" of the cron job * Use default timeout in get_param directly * OCA Transbot updated translations from Transifex * [IMP] count all child processes' resources * [FIX] only look at parent if workers are configured * [ADD] tell the server that we don't know a value * [FIX] pass timeout as keyword argument * [ADD] hints for coveralls * OCA Transbot updated translations from Transifex * OCA Transbot updated translations from Transifex * OCA Transbot updated translations from Transifex * OCA Transbot updated translations from Transifex * OCA Transbot updated translations from Transifex * OCA Transbot updated translations from Transifex * [ADD] migrate dead_mans_switch_client * [IMP] better readme, s/8.0/9.0/
Please check Travis and runbot status |
Sorry @pedrobaeza I started with just the patch-commits, so those were destined to fail. I've now migrated the code, so Travis is giving the green light, but Runbot seems to have failed. I was able to open and log into the Runbot, and it seemed to work fine. I wasn't able to download any log files. What can I do to fix the runbot error? |
You can check here the logs: https://runbot2.odoo-community.org/runbot/static/build/3327863-1277-e764df/logs/job_20_test_all.txt It's a warning as ir.todo has changed its structure:
|
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.
Nice 👍
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.
Small detail. Need to have 0 warnings during build.
- Model ir.cron has changed slightly - urllib2 no longer exists in Python 3 - Strings given to urllib.requests.Request must be encoded as bytes
e764dff
to
a748e4a
Compare
This still seems to be hanging around. Is there anyone who would be willing to be a second reviewer? Does @pedrobaeza know? |
OK, let's go for an optimistic merging. |
No description provided.