-
Notifications
You must be signed in to change notification settings - Fork 199
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
New traceback with 0.2.5 - error: integer out of range for 'L' format code #557
Comments
Only seeing this on a few hosts so far. |
Most likely they're 32 bit hosts.. I've quite literally just pushed 0.2.6 to PyPI, it includes a fix :) https://mitogen.readthedocs.io/en/latest/changelog.html#v0-2-6-2019-03-06 Thanks a ton for reporting this anyway! And please let me know if it's still broken for you in 0.2.6. |
They are x86_64 hosts, but I'll try with 0.2.6 |
In 0.2.6 those fields were forced to be signed 64-bit, so no munging should be required for the thread ID or instance ID (both of which might be negative after IIRC Python applies its hash randomization). But this hadn't manifested on 64-bit before. Are they Intel machines? It's possible it was pure luck this only turned up on ARM the first time around |
This might be an entirely different issue -- it's not Latch, it's subprocess. Please don't waste time retesting yet, this looks like something different to what I just said :( |
It's the process affinity code |
How many CPUs in the machines that failed? |
80 |
Hi Orion, This is on master. There should be another release within weeks. Apologies for the inconvenience, and please let me know if you have any more issues. Thanks again! |
This patch seems to have done the trick. Thanks for the extremely fast fix. |
* origin/v027: docs: update Changelog for release. Bump version for release. issue #587: update Changelog issue #587: ansible: descriptive version check during startup. ansible: descriptive version check during startup. issue #581: expose mitogen_mask_remote_name variable. issue #576: fix Kwargs minor version check. issue #574: fix ISSUE_TEMPLATE link issue #575: fix exception text rendering docs: remove infringing mark docs: fix config var scope docs: faster stats-preserving redirect docs: update ansible page issue #570: add firewalld to always-fork list for now. docs: removed excess word docs: fixed message routing example description docs: removed repeated word docs: update Changelog; closes #557. issue #557: support correct cpu_set_t size
Related to mitogen-hq/mitogen#557: the cookie format was only suitable for 64 bit source hosts.
Which version of Ansible are you running?
ansible-2.7.8-1.el7.noarch
Have you tried the latest master version from Git?
no
Mention your host and target OS and versions
ansible-pull on CentOS 7.6
This is new with 0.2.5:
The text was updated successfully, but these errors were encountered: