Bug 33198 - Searching in processes for 'all' yields strange results
Searching in processes for 'all' yields strange results
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - Process overview
UCS 4.1
Other Linux
: P5 minor (vote)
: UCS 4.3
Assigned To: Jannik Ahlers
Florian Best
: interim-2
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-11-07 14:18 CET by Dirk Wiesenthal
Modified: 2018-03-14 14:38 CET (History)
3 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 2: Improvement: Would be a product improvement
Who will be affected by this bug?: 1: Will affect a very few installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.023
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Usability
Max CVSS v3 score:


Attachments
Finding all kind of unrelated processes (25.37 KB, image/png)
2013-11-07 14:18 CET, Dirk Wiesenthal
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Wiesenthal univentionstaff 2013-11-07 14:18:28 CET
Created attachment 5576 [details]
Finding all kind of unrelated processes

Is is not very intuitive what is returned when searching for 'all'. Reason is

if category == 'all':
  for value in listEntry.itervalues():
    if pattern.match(str(value)):

and listEntry has keys like 'mem', 'cpu', 'timestamp' and so on. I would expect searching in: user, pid, command (the choices besides 'all').
Comment 1 Jannik Ahlers univentionstaff 2018-01-11 09:40:48 CET
I reduced the keys searched in to only the relevant ones: user, pid, command.
see branch jahlers/33198-top-search
Comment 2 Florian Best univentionstaff 2018-01-11 17:01:03 CET
That's cool!
Comment 3 Florian Best univentionstaff 2018-01-22 10:39:59 CET
Please merge into UCS 4.3 and create a changelog entry in doc/changelog/changelog-4.3-0.xml.
Comment 4 Jannik Ahlers univentionstaff 2018-01-22 12:53:47 CET
changelog-4.3-0.xml
916cd75af41b | Bug #33198: changelog

univention-management-console-module-top (7.0.1-3)
5182b21b8a3e | Bug #33198: Merge branch 'jahlers/33198-top-search' into 4.3-0
9b74967b09bf | Bug #33198: Better results in the process overview UMC module's search function
Comment 5 Florian Best univentionstaff 2018-01-22 21:16:20 CET
The package was only imported in our build system but not build. I build it.

I adjusted the formatation of the Changelog entry:
https://git.knut.univention.de/univention/ucs/commit/fc3ca64603b35898372eb2cee88482ef0619fd06
Comment 6 Florian Best univentionstaff 2018-01-23 10:42:17 CET
OK: the seach is optimized and don't find values from CPU usage or memory use anymore.
OK: Changelog entry
Comment 7 Stefan Gohmann univentionstaff 2018-03-14 14:38:14 CET
UCS 4.3 has been released:
 https://docs.software-univention.de/release-notes-4.3-0-en.html
 https://docs.software-univention.de/release-notes-4.3-0-de.html

If this error occurs again, please use "Clone This Bug".