Bug 36358 - modules/top/__init__.py: kill() takes exactly 1 argument (2 given)
modules/top/__init__.py: kill() takes exactly 1 argument (2 given)
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - Process overview
UCS 3.2
Other Linux
: P5 normal (vote)
: UCS 3.2-4-errata
Assigned To: Florian Best
Johannes Keiser
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-10-31 01:21 CET by Dirk Wiesenthal
Modified: 2014-12-19 13:59 CET (History)
4 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Wiesenthal univentionstaff 2014-10-31 01:21:20 CET
Also happens when using python-psutil from Debian Backports (part of Pulse app).

Fix needs to be backported. Maybe one can get __version__ of the module (0.7.* should be the newer version). Otherwise duck typing.

+++ This bug was initially created as a clone of Bug #36161 +++

Execution of command 'top/kill' has failed:

Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.7/univention/management/console/modules/__init__.py", line 177, in _decorated
    return function(self, request, *args, **kwargs)
  File "/usr/lib/pymodules/python2.7/univention/management/console/modules/top/__init__.py", line 111, in kill
    process.kill(15)
TypeError: kill() takes exactly 1 argument (2 given)

Found by /usr/share/ucs-test/60_umc-system/22_umc-service-proc-killing
Comment 1 Florian Best univentionstaff 2014-12-04 11:27:59 CET
only process.kill() and process.terminate() are affected:

>>> import psutil, os
>>> process = psutil.Process(os.getpid())
>>> print process.username, process.pid, process.get_memory_percent(), process.cmdline, process.name, process.get_cpu_times()

Fix: svn r56446
YAML: 2014-12-04-univention-management-console-module-top.yaml
Package: univention-management-console-module-top (3.0.3-2)
Comment 2 Florian Best univentionstaff 2014-12-05 13:07:58 CET
QA: Please kill and terminate a process first. Then install the pulse app. Kill and terminate a process again. If this works you can set the Bug to VERIFIED.
Comment 3 Johannes Keiser univentionstaff 2014-12-17 19:36:07 CET
Fix: OK
YAML: OK
Comment 4 Janek Walkenhorst univentionstaff 2014-12-19 13:59:13 CET
http://errata.univention.de/ucs/3.2/259.html