-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
disk_io_counters() fails on Linux kernel 4.19 #1354
Comments
Yes, I got the same ValueError. |
Just for reference, last time we had a similar issue was #767.
|
….18+ Linux kernel 4.18+ added 4 fields, ingore them and parse the rest as usual.
Can we get a point release with this fix? Or will there be a new release soon? |
Latest version on PYPI already includes this fix. ;) |
@giampaolo I've tried latest on PyPI and unfortunately I still get the same error:
|
Sorry, this should have been fixed in 8f99f37 which is still not released. If you try latest GIT version it should work. |
@giampaolo yes, latest git version works. Do you have any plans on releasing a new version to PyPI? |
temporary fix for python 2.7:
|
@giampaolo when do you cut the next release? |
I am traveling now. Please be patient. :)
On Fri, 18 Jan 2019 at 15:58, rmoriz ***@***.***> wrote:
@giampaolo <https://github.com/giampaolo> we do you plan to cut the next
release?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1354 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAplLLDQFYHtaVjodQTzHQfmfWyWjmchks5vEeD9gaJpZM4X-0d1>
.
--
Giampaolo - http://grodola.blogspot.com
|
…or r=ahal Imports giampaolo/psutil@8f99f37 to fix a bug with Linux kernel 4.18+, giampaolo/psutil#1354 (to be included in a future release of psutil, version 5.5.0) Differential Revision: https://phabricator.services.mozilla.com/D16028 --HG-- extra : moz-landing-system : lando
…or r=ahal Imports giampaolo/psutil@8f99f37 to fix a bug with Linux kernel 4.18+, giampaolo/psutil#1354 (to be included in a future release of psutil, version 5.5.0) Differential Revision: https://phabricator.services.mozilla.com/D16028
* origin/master: (182 commits) giampaolo#1394 / windows / process exe(): convert errno 0 into ERROR_ACCESS_DENIED; errno 0 occurs when the Python process runs in 'Virtual Secure Mode' pre-release fix win num_handles() test update readme fix giampaolo#1111: use a lock to make Process.oneshot() thread safe pdate HISTORY giampaolo#1373: different approach to oneshot() cache (pass Process instances around - which is faster) use PROCESS_QUERY_LIMITED_INFORMATION also for username() Linux: refactor _parse_stat_file() and return a dict instead of a list (+ maintainability) fix giampaolo#1357: do not expose Process' memory_maps() and io_counters() methods if not supported by the kernel giampaolo#1376 Windows: check if variable is NULL before free()ing it enforce lack of support for Win XP fix giampaolo#1370: improper usage of CloseHandle() may lead to override the original error code resulting in raising a wrong exception update HISTORY (Windows) use PROCESS_QUERY_LIMITED_INFORMATION access rights (giampaolo#1376) update HISTORY revert 5398c48; let's do it in a separate branch giampaolo#1111 make Process.oneshot() thread-safe sort HISTORY give CREDITS to @EccoTheFlintstone for giampaolo#1368 fix ionice set not working on windows x64 due to LENGTH_MISMATCH (giampaolo#1368) make flake8 happy give CREDITS to @amanusk for giampaolo#1369 / giampaolo#1352 and update doc Add CPU frequency support for FreeBSD (giampaolo#1369) giampaolo#1359: add test case for cpu_count(logical=False) against lscpu utility disable false positive mem test on travis + osx fix PEP8 style mistakes give credits to @koenkooi for giampaolo#1360 Fix giampaolo#1354 [Linux] disk_io_counters() fails on Linux kernel 4.18+ (giampaolo#1360) giampaolo#1350: give credits to @amanusk FreeBSD adding temperature sensors (WIP) (giampaolo#1350) pre release sensors_temperatures() / linux: convert defaultdict to dict fix giampaolo#1004: Process.io_counters() may raise ValueError fix giampaolo#1307: [Linux] disk_partitions() does not honour PROCFS_PATH refactor hasattr() checks as global constants giampaolo#1197 / linux / cpu_freq(): parse /proc/cpuinfo in case /sys/devices/system/cpu fs is not available fix giampaolo#1277 / osx / virtual_memory: 'available' and 'used' memory were not calculated properly travis / osx: set py 3.6 travis: disable pypy; se py 3.7 on osx skip test on PYPY + Travis fix travis fix giampaolo#715: do not print exception on import time in case cpu_times() fails. fix different travis failures give CREDITS for giampaolo#1320 to @truthbk [aix] improve compilation on AIX, better support for gcc/g++ + fix cpu metrics (giampaolo#1320) give credits to @alxchk for giampaolo#1346 (sunOS) Fix giampaolo#1346 (giampaolo#1347) giampaolo#1284, giampaolo#1345 - give credits to @amanusk Add parsing for /sys/class/thermal (giampaolo#1345) Fix decoding error in tests catch UnicodeEncodeError on print() use memory tolerance in occasionally failing test Fix random 0xC0000001 errors when querying for Connections (giampaolo#1335) Correct capitalization of PyPI (giampaolo#1337) giampaolo#1341: move open() utilities/wrappers in _common.py Refactored ps() function in test_posix (giampaolo#1341) fix giampaolo#1343: document Process.as_dict() attrs values giampaolo#1332 - update HISTORY make psutil_debug() aware of PSUTIL_DEBUG (giampaolo#1332) also include PYPY (or try to :P) travis: add python 3.7 build add download badge remove failing test assertions remove failing test make test more robust pre release pre release set version to 5.4.7 OSX / SMC / sensors: revert giampaolo#1284 (giampaolo#1325) setup.py: add py 3.7 fix giampaolo#1323: [Linux] sensors_temperatures() may fail with ValueError fix failing linux tests giampaolo#1321 add unit tests giampaolo#1321: refactoring make disk_io_counters more robust (giampaolo#1324) fix typo Fix DeprecationWarning: invalid escape sequence (giampaolo#1318) remove old test update is_storage_device() docstring fix giampaolo#1305 / disk_io_counters() / Linux: assume SECTOR_SIZE is a fixed 512 giampaolo#1313 remove test which no longer makes sense disk_io_counters() - linux: mimic iostat behavior (giampaolo#1313) fix wrong reference link in doc disambiguate TESTFN for parallel testing fix giampaolo#1309: add STATUS_PARKED constant and fix STATUS_IDLE (both on linux) give CREDITS to @sylvainduchesne for giampaolo#1294 retain GIL when querying connections table (giampaolo#1306) Update index.rst (giampaolo#1308) fix giampaolo#1279: catch and skip ENODEV in net_if_stat() appveyor: retire 3.5, add 3.7 revert file renaming of macos files; get them back to 'osx' prefix winmake: add upload-wheels cmd Rename OSX to macOS (giampaolo#1298) apveyor: reset py 3.4 and remove 3.7 (not available yet) try to fix occasional children() failure on Win: https://ci.appveyor.com/project/giampaolo/psutil/build/job/je3qyldbb86ff66h appveyor: remove py 3.4 and add 3.7 giampaolo#1284: give credits to @amanusk + some minor adjustments little refactoring Osx temps (giampaolo#1284) ...
On debian stretch (using psutil 5.0.1 with python3), the following workaround works on kernel 5.0.8 :
|
…or. r=ahal, a=jcristau DONTBUILD Imports giampaolo/psutil@8f99f37 to fix a bug with Linux kernel 4.18+, giampaolo/psutil#1354 (to be included in a future release of psutil, version 5.5.0) Differential Revision: https://phabricator.services.mozilla.com/D16028 [jcristau: re-applying this change which was accidentally reverted in https://hg.mozilla.org/releases/mozilla-release/rev/e2030db2e502]
…or. r=ahal, a=jcristau DONTBUILD Imports giampaolo/psutil@8f99f37 to fix a bug with Linux kernel 4.18+, giampaolo/psutil#1354 (to be included in a future release of psutil, version 5.5.0) Differential Revision: https://phabricator.services.mozilla.com/D16028 [jcristau: re-applying this change which was accidentally reverted in https://hg.mozilla.org/releases/mozilla-release/rev/e2030db2e502] --HG-- extra : source : e26eda77b71c6c02db69441236dd95fc0db5c7f2
…or r=ahal Imports giampaolo/psutil@8f99f37 to fix a bug with Linux kernel 4.18+, giampaolo/psutil#1354 (to be included in a future release of psutil, version 5.5.0) Differential Revision: https://phabricator.services.mozilla.com/D16028 UltraBlame original commit: e26eda77b71c6c02db69441236dd95fc0db5c7f2
…or r=ahal Imports giampaolo/psutil@8f99f37 to fix a bug with Linux kernel 4.18+, giampaolo/psutil#1354 (to be included in a future release of psutil, version 5.5.0) Differential Revision: https://phabricator.services.mozilla.com/D16028 UltraBlame original commit: e26eda77b71c6c02db69441236dd95fc0db5c7f2
…or r=ahal Imports giampaolo/psutil@8f99f37 to fix a bug with Linux kernel 4.18+, giampaolo/psutil#1354 (to be included in a future release of psutil, version 5.5.0) Differential Revision: https://phabricator.services.mozilla.com/D16028 UltraBlame original commit: e26eda77b71c6c02db69441236dd95fc0db5c7f2
The 4.18 kernel introduced a number of new fields into the disk io stats proc fileystem, and running tests on new versions of linux is causing failures because psutil cannot interpret the new fields. This fix updates the version such that it works w/ newer kernels. Relates giampaolo/psutil#1354
The 4.18 kernel introduced a number of new fields into the disk io stats proc fileystem, and running tests on new versions of linux is causing failures because psutil cannot interpret the new fields. This fix updates the version such that it works w/ newer kernels. Relates giampaolo/psutil#1354
When running on
4.19.0-1.el7.elrepo.x86_6
, callingdisk_io_counters()
raises ValueError:The text was updated successfully, but these errors were encountered: