Bug 427548 - Process table for remote machine shows local machine
Summary: Process table for remote machine shows local machine
Status: REPORTED
Alias: None
Product: ksysguard
Classification: Applications
Component: Process Controller - krunner part (show other bugs)
Version: 5.19.5
Platform: Other Linux
: NOR major
Target Milestone: ---
Assignee: KSysGuard Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-10-11 15:27 UTC by Paul Worrall
Modified: 2021-06-17 15:22 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
Showing remote and local process list (258.04 KB, image/png)
2021-06-17 14:44 UTC, Martin Sandsmark 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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.