Guru Meditation Reports broken with psutil

[stanzgy]

Many gmr codes which invokes psutil are broken with lastest pstutil 2.2.1.

examples:
https://github.com/openstack/oslo-incubator/blob/master/openstack/common/report/models/process.py#L41

self['uids'] = {'real': process.uids.real,
    'effective': process.uids.effective,
    'saved': process.uids.saved}

try with ipython:

In [1]: import psutil

In [2]: a=psutil.Process(15097)

In [3]: a.uids
Out[3]: <bound method Process.uids of <psutil.Process(pid=15097, name='designate-centr') at 139686329973584>>

In [4]: a.uids.real
---------------------------------------------------------------------------
AttributeError Traceback (most recent call last)
<ipython-input-4-ef3b965e8ef8> in <module>()
----> 1 a.uids.real

AttributeError: 'function' object has no attribute 'real'

In [5]: a.uids().real
Out[5]: 1000

[stanzgy (stanzgy)]

psutil 2.0.0 which matches psutil>=1.1.1,<2.0.0 in global-requirement.txt also has the problem

[Davanum Srinivas (DIMS) (dims-v)]

looks like they changed the apis:

https://github.com/giampaolo/psutil/commit/0bb597bf4d91cf60b47542bde4daa78eb4dbfa51

[Doug Hellmann (doug-hellmann)]

@stanzgy - That requirement specification says "<2.0.0" so it should exclude the newer versions of the library.

[Doug Hellmann (doug-hellmann)]

Related change in the incubator: https://review.openstack.org/#/c/162945/

[Doug Hellmann (doug-hellmann)]

To upgrade to the newer psutils we will actually need a version of our report module that supports both APIs. Then we can push that change out to the projects, update the global requirement, graduate the report library, raise its minimum version, update the projects to use it, then update the global requirements minimum version.

[Davanum Srinivas (DIMS) (dims-v)]

We should fix this in oslo.reports

[Davanum Srinivas (DIMS) (dims-v)]

https://review.openstack.org/#/c/198578/