Bug 427548

Summary: Process table for remote machine shows local machine
Product: [Applications] ksysguard Reporter: Paul Worrall <p.r.worrall>
Component: Process Controller - krunner partAssignee: KSysGuard Developers <ksysguard-bugs>
Status: REPORTED ---    
Severity: major CC: kde, martin.sandsmark, plasma-bugs
Priority: NOR    
Version: 5.19.5   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: Showing remote and local process list

Description Paul Worrall 2020-10-11 15:27:56 UTC 
SUMMARY
Unable to create a tab in ksysguard that shows the process table of a remote machine

STEPS TO REPRODUCE
1. Create a new 1x1 Tab
2. Select "Monitor Remote Machine..." and connect via ssh
3. Drag "Process Controller (Process Controller)" from remote machine sensor browser onto new Tab

OBSERVED RESULT
Process table of localhost is displayed

EXPECTED RESULT
Process table of remote machine is displayed

SOFTWARE/OS VERSIONS
Windows: N/A
macOS: N/A
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 5.19.5
KDE Frameworks Version: 5.74.0
Qt Version: 5.15.0

ADDITIONAL INFORMATION
Comment 1 Martin Sandsmark 2021-06-17 14:44:55 UTC 
Created attachment 139435 [details]
Showing remote and local process list
Comment 2 Martin Sandsmark 2021-06-17 14:46:05 UTC 
Only setting it as major even if it is completely broken, since I don't think remote monitoring is one of the key features.
Comment 3 Martin Sandsmark 2021-06-17 15:02:54 UTC 
064688415a1e6db98a3fa20690320596a4f6d3da seems to be the culprit, so adding David to CC.

For some reason it now always passes an empty string as hostname to the Processes constructor, so it will never even try to create a KSysGuard::ProcessesRemote.
Comment 4 Martin Sandsmark 2021-06-17 15:06:55 UTC 
There seems to have been some major refactorings since then with some bad assumptions (e. g. the ExtendedProcesses is for some reason a singleton), so it isn't trivial to fix either.